[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 0/2] vhost-user-gpu get_edid feature
|
From: |
Erico Nunes |
|
Subject: |
[PATCH 0/2] vhost-user-gpu get_edid feature |
|
Date: |
Thu, 11 May 2023 14:58:01 +0200 |
This adds support to the virtio-gpu get_edid command when using the
vhost-user-gpu implementation in contrib/.
So far, qemu has been outputting the following message:
EDID requested but the backend doesn't support it.
when using that implementation.
This is tested with vhost-user-gpu, the dbus ui backend and the
monitor-edid application, which now shows complete "QEMU Monitor" edid
data.
In this v1, I would appreciate some feedback especially regarding:
- Can we enable it by default or do need to create another config option
flag for it?
- Can we now also remove the "EDID requested but the backend doesn't
support it." warning and logic from hw/display or do we still want to
keep that around for other potential implementations of
vhost-user-gpu?
- The structs used as payloads of the vhost-user-gpu messages. Looks
like there was no command so far requiring bidirectional messages with
different payloads so I just based it on similar available ones.
Thanks
Erico Nunes (2):
virtio-gpu: refactor generate_edid function to virtio_gpu_base
vhost-user-gpu: implement get_edid feature
contrib/vhost-user-gpu/vhost-user-gpu.c | 53 ++++++++++++++++++++++++-
contrib/vhost-user-gpu/virgl.c | 3 ++
contrib/vhost-user-gpu/vugpu.h | 8 ++++
docs/interop/vhost-user-gpu.rst | 9 +++++
hw/display/vhost-user-gpu.c | 31 +++++++++++++++
hw/display/virtio-gpu-base.c | 17 ++++++++
hw/display/virtio-gpu.c | 20 +---------
include/hw/virtio/virtio-gpu.h | 2 +
8 files changed, 122 insertions(+), 21 deletions(-)
--
2.39.2
- [PATCH 0/2] vhost-user-gpu get_edid feature,
Erico Nunes <=