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

Antonio Caggiano posted 3 patches 4 years, 4 months ago
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210928131548.72005-1-antonio.caggiano@collabora.com
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>
There is a newer version of this series
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(-)
[RFC 0/3] virtio-gpu: Shared memory capability
Posted by Antonio Caggiano 4 years, 4 months ago
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(-)

-- 
2.30.2


Re: [RFC 0/3] virtio-gpu: Shared memory capability
Posted by Gerd Hoffmann 4 years, 4 months ago
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