[Qemu-commits] [qemu/qemu] 93abfc: vga: set owner for mmio regions

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 93abfc88bd649de1933588bfc7175605331b3ea9
      
https://github.com/qemu/qemu/commit/93abfc88bd649de1933588bfc7175605331b3ea9
  Author: Gerd Hoffmann <address@hidden>
  Date:   2018-06-26 (Tue, 26 Jun 2018)

  Changed paths:
    M hw/display/vga-pci.c
    M hw/display/vga_int.h
    M hw/display/virtio-vga.c

  Log Message:
  -----------
  vga: set owner for mmio regions

This makes sure the regions are properly cleaned when unplugging -device
seconday-vga.

Reported-by: Thomas Huth <address@hidden>
Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Thomas Huth <address@hidden>
Message-id: address@hidden


  Commit: 6b9b3c1e303cc0dda99e326031d2b1b612b80e3b
      
https://github.com/qemu/qemu/commit/6b9b3c1e303cc0dda99e326031d2b1b612b80e3b
  Author: Gerd Hoffmann <address@hidden>
  Date:   2018-06-26 (Tue, 26 Jun 2018)

  Changed paths:
    M hw/display/ramfb.c

  Log Message:
  -----------
  ramfb: fix overflow

> CID 1393621:    (OVERFLOW_BEFORE_WIDEN)
> Potentially overflowing expression "stride * s->height" with type "unsigned
> int" (32 bits, unsigned) is evaluated using +32-bit arithmetic, and then used
> in a context that expects an expression of type "hwaddr" (64 bits, unsigned).

Fix by changing stride from uint32_t to hwaddr.

Signed-off-by: Gerd Hoffmann <address@hidden>
Message-id: address@hidden


  Commit: fd1cfb875be551064a27d909edb406316f011f2c
      
https://github.com/qemu/qemu/commit/fd1cfb875be551064a27d909edb406316f011f2c
  Author: Peter Maydell <address@hidden>
  Date:   2018-06-26 (Tue, 26 Jun 2018)

  Changed paths:
    M include/hw/virtio/virtio-gpu.h

  Log Message:
  -----------
  virtio-gpu-3d: Drop workaround for VIRTIO_GPU_CAPSET_VIRGL2 define

In commit a8bff79e9f27df we added a definition to hw/virtio/virtio-gpu.h
for VIRTIO_GPU_CAPSET_VIRGL2, as a workaround for it not yet being
in the Linux kernel headers. In commit 77d361b13c19 we updated our
kernel headers to a version which does define the macro, so we can
now remove our workaround.

Signed-off-by: Peter Maydell <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>


  Commit: 7e2d08863b5cd4d35e6e0dc211b0cceaa350ade2
      
https://github.com/qemu/qemu/commit/7e2d08863b5cd4d35e6e0dc211b0cceaa350ade2
  Author: Peter Maydell <address@hidden>
  Date:   2018-06-26 (Tue, 26 Jun 2018)

  Changed paths:
    M hw/display/ramfb.c
    M hw/display/vga-pci.c
    M hw/display/vga_int.h
    M hw/display/virtio-vga.c
    M include/hw/virtio/virtio-gpu.h

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/kraxel/tags/vga-20180626-pull-request' 
into staging

vga: bugfix collection.

# gpg: Signature made Tue 26 Jun 2018 17:27:23 BST
# gpg:                using RSA key 4CB6D8EED3E87138
# gpg: Good signature from "Gerd Hoffmann (work) <address@hidden>"
# gpg:                 aka "Gerd Hoffmann <address@hidden>"
# gpg:                 aka "Gerd Hoffmann (private) <address@hidden>"
# Primary key fingerprint: A032 8CFF B93A 17A7 9901  FE7D 4CB6 D8EE D3E8 7138

* remotes/kraxel/tags/vga-20180626-pull-request:
  virtio-gpu-3d: Drop workaround for VIRTIO_GPU_CAPSET_VIRGL2 define
  ramfb: fix overflow
  vga: set owner for mmio regions

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/0ab4c574a554...7e2d08863b5c
      **NOTE:** This service been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.