mbox series

[RFC,0/3] virtio-gpu: Shared memory capability

Message ID 20210928131548.72005-1-antonio.caggiano@collabora.com (mailing list archive)
Headers show
Series virtio-gpu: Shared memory capability | expand

Message

Antonio Caggiano Sept. 28, 2021, 1:15 p.m. UTC 
Previously part of [0], now a patch series on its own.

This patch series cherry picks two commits from [1] and applies one fix
according to [2], which should answer Gerd's comment [3] on previous
patch.

RFC as I believe it needs a proper review from both authors of first two
commits.

[0] https://www.mail-archive.com/qemu-devel@nongnu.org/msg826897.html
[1] https://gitlab.freedesktop.org/virgl/qemu/-/commits/virtio-gpu-next/
[2] https://github.com/torvalds/linux/commit/0dd4ff93f4c8dba016ad79384007da4938cd54a1
[3] https://www.mail-archive.com/qemu-devel@nongnu.org/msg827306.html

Antonio Caggiano (1):
  virtio: Fix shared memory capability

Dr. David Alan Gilbert (1):
  virtio: Add shared memory capability

Gerd Hoffmann (1):
  virtio-gpu: hostmem

 hw/display/virtio-gpu-pci.c    | 14 ++++++++++++++
 hw/display/virtio-gpu.c        |  1 +
 hw/display/virtio-vga.c        | 32 +++++++++++++++++++++++---------
 hw/virtio/virtio-pci.c         | 19 +++++++++++++++++++
 hw/virtio/virtio-pci.h         |  4 ++++
 include/hw/virtio/virtio-gpu.h |  5 +++++
 6 files changed, 66 insertions(+), 9 deletions(-)

Comments

Gerd Hoffmann Sept. 29, 2021, 7:43 a.m. UTC | #1 
On Tue, Sep 28, 2021 at 03:15:45PM +0200, Antonio Caggiano wrote:
> Previously part of [0], now a patch series on its own.
> 
> This patch series cherry picks two commits from [1] and applies one fix
> according to [2], which should answer Gerd's comment [3] on previous
> patch.
> 
> RFC as I believe it needs a proper review from both authors of first two
> commits.

Looks sane on a quick glance.  I'd tend to squash patch #3 into patch #2.
Doesn't make much sense standalone though ...

take care,
  Gerd