[libvirt] [PATCH v2 14/16] qemu: build vhost-user GPU devices
Ján Tomko
jtomko at redhat.com
Tue Aug 27 09:17:18 UTC 2019
On Fri, Aug 23, 2019 at 12:21:58PM -0400, Cole Robinson wrote:
>From: Marc-André Lureau <marcandre.lureau at redhat.com>
>
>For each vhost-user GPUs,
>- build a socket chardev, and pass the vhost-user socket to it
>- build a vhost-user video device and associate it with the chardev
>
>Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
>Signed-off-by: Cole Robinson <crobinso at redhat.com>
>---
> src/qemu/qemu_command.c | 46 +++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 44 insertions(+), 2 deletions(-)
>
>diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
>index 8bef103f68..0e1d9510e5 100644
>--- a/src/qemu/qemu_command.c
>+++ b/src/qemu/qemu_command.c
>@@ -4672,8 +4672,15 @@ qemuBuildDeviceVideoStr(const virDomainDef *def,
> goto error;
> }
>
>- if (STREQ(model, "virtio-gpu")) {
>- if (qemuBuildVirtioDevStr(&buf, "virtio-gpu", qemuCaps,
Eww, why do we do a string comparison here when we store the model as an
enum?
>+ if (video->vhostuser) {
>+ if (STREQ(model, "virtio-vga"))
>+ model = "vhost-user-vga";
>+ if (STREQ(model, "virtio-gpu"))
>+ model = "vhost-user-gpu";
>+ }
Not sure why we need to reassign model here instead of getting it right
the first time.
>+
How different is the vhost-user-gpu device from virtio-gpu,
don't we new a new VIDEO_TYPE_VHOST_USER instead?
if (video->type == VIRTIO && !primary) for the condition below:
>+ if (STREQ(model, "virtio-gpu") || STREQ(model, "vhost-user-gpu")) {
>+ if (qemuBuildVirtioDevStr(&buf, model, qemuCaps,
> VIR_DOMAIN_DEVICE_VIDEO, video) < 0) {
> goto error;
> }
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190827/fa70a818/attachment-0001.sig>
More information about the libvir-list
mailing list