mbox series

[v12,0/9] rutabaga_gfx + gfxstream

Message ID 20230824234046.499-1-gurchetansingh@chromium.org (mailing list archive)
Headers show
Series rutabaga_gfx + gfxstream | expand

Message

Gurchetan Singh Aug. 24, 2023, 11:40 p.m. UTC 
From: Gurchetan Singh <gurchetansingh@google.com>

Prior versions:

Changes since v11:
- Incorporated review feedback

How to build both rutabaga and gfxstream guest/host libs:

https://crosvm.dev/book/appendix/rutabaga_gfx.html

Branch containing this patch series (now on QEMU Gitlab):

https://gitlab.com/gurchetansingh/qemu/-/commits/qemu-gfxstream-v12

Antonio Caggiano (2):
  virtio-gpu: CONTEXT_INIT feature
  virtio-gpu: blob prep

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

Gerd Hoffmann (1):
  virtio-gpu: hostmem

Gurchetan Singh (5):
  gfxstream + rutabaga prep: added need defintions, fields, and options
  gfxstream + rutabaga: add initial support for gfxstream
  gfxstream + rutabaga: meson support
  gfxstream + rutabaga: enable rutabaga
  docs/system: add basic virtio-gpu documentation

 docs/system/device-emulation.rst     |    1 +
 docs/system/devices/virtio-gpu.rst   |  112 +++
 hw/display/meson.build               |   22 +
 hw/display/virtio-gpu-base.c         |    6 +-
 hw/display/virtio-gpu-pci-rutabaga.c |   47 ++
 hw/display/virtio-gpu-pci.c          |   14 +
 hw/display/virtio-gpu-rutabaga.c     | 1119 ++++++++++++++++++++++++++
 hw/display/virtio-gpu.c              |   16 +-
 hw/display/virtio-vga-rutabaga.c     |   50 ++
 hw/display/virtio-vga.c              |   33 +-
 hw/virtio/virtio-pci.c               |   18 +
 include/hw/virtio/virtio-gpu-bswap.h |   15 +
 include/hw/virtio/virtio-gpu.h       |   41 +
 include/hw/virtio/virtio-pci.h       |    4 +
 meson.build                          |    7 +
 meson_options.txt                    |    2 +
 scripts/meson-buildoptions.sh        |    3 +
 softmmu/qdev-monitor.c               |    3 +
 softmmu/vl.c                         |    1 +
 19 files changed, 1495 insertions(+), 19 deletions(-)
 create mode 100644 docs/system/devices/virtio-gpu.rst
 create mode 100644 hw/display/virtio-gpu-pci-rutabaga.c
 create mode 100644 hw/display/virtio-gpu-rutabaga.c
 create mode 100644 hw/display/virtio-vga-rutabaga.c

Comments

Akihiko Odaki Aug. 25, 2023, 4:53 a.m. UTC | #1 
On 2023/08/25 8:40, Gurchetan Singh wrote:
> From: Gurchetan Singh <gurchetansingh@google.com>
> 
> Prior versions:
> 
> Changes since v11:
> - Incorporated review feedback
> 
> How to build both rutabaga and gfxstream guest/host libs:
> 
> https://crosvm.dev/book/appendix/rutabaga_gfx.html
> 
> Branch containing this patch series (now on QEMU Gitlab):
> 
> https://gitlab.com/gurchetansingh/qemu/-/commits/qemu-gfxstream-v12
> 
> Antonio Caggiano (2):
>    virtio-gpu: CONTEXT_INIT feature
>    virtio-gpu: blob prep
> 
> Dr. David Alan Gilbert (1):
>    virtio: Add shared memory capability
> 
> Gerd Hoffmann (1):
>    virtio-gpu: hostmem
> 
> Gurchetan Singh (5):
>    gfxstream + rutabaga prep: added need defintions, fields, and options
>    gfxstream + rutabaga: add initial support for gfxstream
>    gfxstream + rutabaga: meson support
>    gfxstream + rutabaga: enable rutabaga
>    docs/system: add basic virtio-gpu documentation
> 
>   docs/system/device-emulation.rst     |    1 +
>   docs/system/devices/virtio-gpu.rst   |  112 +++
>   hw/display/meson.build               |   22 +
>   hw/display/virtio-gpu-base.c         |    6 +-
>   hw/display/virtio-gpu-pci-rutabaga.c |   47 ++
>   hw/display/virtio-gpu-pci.c          |   14 +
>   hw/display/virtio-gpu-rutabaga.c     | 1119 ++++++++++++++++++++++++++
>   hw/display/virtio-gpu.c              |   16 +-
>   hw/display/virtio-vga-rutabaga.c     |   50 ++
>   hw/display/virtio-vga.c              |   33 +-
>   hw/virtio/virtio-pci.c               |   18 +
>   include/hw/virtio/virtio-gpu-bswap.h |   15 +
>   include/hw/virtio/virtio-gpu.h       |   41 +
>   include/hw/virtio/virtio-pci.h       |    4 +
>   meson.build                          |    7 +
>   meson_options.txt                    |    2 +
>   scripts/meson-buildoptions.sh        |    3 +
>   softmmu/qdev-monitor.c               |    3 +
>   softmmu/vl.c                         |    1 +
>   19 files changed, 1495 insertions(+), 19 deletions(-)
>   create mode 100644 docs/system/devices/virtio-gpu.rst
>   create mode 100644 hw/display/virtio-gpu-pci-rutabaga.c
>   create mode 100644 hw/display/virtio-gpu-rutabaga.c
>   create mode 100644 hw/display/virtio-vga-rutabaga.c
> 

Thanks for keeping working on this. For the entire series:
Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Gurchetan Singh Aug. 25, 2023, 7:18 p.m. UTC | #2 
On Thu, Aug 24, 2023 at 9:53 PM Akihiko Odaki <akihiko.odaki@gmail.com>
wrote:

> On 2023/08/25 8:40, Gurchetan Singh wrote:
> > From: Gurchetan Singh <gurchetansingh@google.com>
> >
> > Prior versions:
> >
> > Changes since v11:
> > - Incorporated review feedback
> >
> > How to build both rutabaga and gfxstream guest/host libs:
> >
> > https://crosvm.dev/book/appendix/rutabaga_gfx.html
> >
> > Branch containing this patch series (now on QEMU Gitlab):
> >
> > https://gitlab.com/gurchetansingh/qemu/-/commits/qemu-gfxstream-v12
> >
> > Antonio Caggiano (2):
> >    virtio-gpu: CONTEXT_INIT feature
> >    virtio-gpu: blob prep
> >
> > Dr. David Alan Gilbert (1):
> >    virtio: Add shared memory capability
> >
> > Gerd Hoffmann (1):
> >    virtio-gpu: hostmem
> >
> > Gurchetan Singh (5):
> >    gfxstream + rutabaga prep: added need defintions, fields, and options
> >    gfxstream + rutabaga: add initial support for gfxstream
> >    gfxstream + rutabaga: meson support
> >    gfxstream + rutabaga: enable rutabaga
> >    docs/system: add basic virtio-gpu documentation
> >
> >   docs/system/device-emulation.rst     |    1 +
> >   docs/system/devices/virtio-gpu.rst   |  112 +++
> >   hw/display/meson.build               |   22 +
> >   hw/display/virtio-gpu-base.c         |    6 +-
> >   hw/display/virtio-gpu-pci-rutabaga.c |   47 ++
> >   hw/display/virtio-gpu-pci.c          |   14 +
> >   hw/display/virtio-gpu-rutabaga.c     | 1119 ++++++++++++++++++++++++++
> >   hw/display/virtio-gpu.c              |   16 +-
> >   hw/display/virtio-vga-rutabaga.c     |   50 ++
> >   hw/display/virtio-vga.c              |   33 +-
> >   hw/virtio/virtio-pci.c               |   18 +
> >   include/hw/virtio/virtio-gpu-bswap.h |   15 +
> >   include/hw/virtio/virtio-gpu.h       |   41 +
> >   include/hw/virtio/virtio-pci.h       |    4 +
> >   meson.build                          |    7 +
> >   meson_options.txt                    |    2 +
> >   scripts/meson-buildoptions.sh        |    3 +
> >   softmmu/qdev-monitor.c               |    3 +
> >   softmmu/vl.c                         |    1 +
> >   19 files changed, 1495 insertions(+), 19 deletions(-)
> >   create mode 100644 docs/system/devices/virtio-gpu.rst
> >   create mode 100644 hw/display/virtio-gpu-pci-rutabaga.c
> >   create mode 100644 hw/display/virtio-gpu-rutabaga.c
> >   create mode 100644 hw/display/virtio-vga-rutabaga.c
> >
>
> Thanks for keeping working on this. For the entire series:
> Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> Tested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
>

Awesome, thanks.  I'll wait a few days for possible additional comments,
otherwise I'll send out v13 with additional r-b tags.