Hi Marc-André,
> -----Original Message-----
> From: marcandre.lur...@redhat.com <marcandre.lur...@redhat.com>
> Sent: Monday, April 15, 2024 4:16 AM
> To: qemu-devel@nongnu.org
> Cc: Kim, Dongwon <dongwon....@intel.com>; dbas...@redhat.com; Marc-
> André Lureau <marcandre.lur...@redhat.com>; Michael S. Tsirkin
> <m...@redhat.com>; Gerd Hoffmann <kra...@redhat.com>
> Subject: [PATCH] vhost-user-gpu: fix import of DMABUF
>
> From: Marc-André Lureau <marcandre.lur...@redhat.com>
>
> When using vhost-user-gpu with GL, qemu -display gtk doesn't show output and
> prints: qemu: eglCreateImageKHR failed
>
> Since commit 9ac06df8b ("virtio-gpu-udmabuf: correct naming of QemuDmaBuf
> size properties"), egl_dmabuf_import_texture() uses backing_{width,height} for
> the texture dimension.
>
> Fixes: commit 9ac06df8b ("virtio-gpu-udmabuf: correct naming of QemuDmaBuf
> size properties")
> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> ---
> hw/display/vhost-user-gpu.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/display/vhost-user-gpu.c b/hw/display/vhost-user-gpu.c index
> 709c8a02a1..baffb1c2d4 100644
> --- a/hw/display/vhost-user-gpu.c
> +++ b/hw/display/vhost-user-gpu.c
> @@ -273,8 +273,8 @@ vhost_user_gpu_handle_display(VhostUserGPU *g,
> VhostUserGpuMsg *msg)
> }
> *dmabuf = (QemuDmaBuf) {
> .fd = fd,
> - .width = m->fd_width,
> - .height = m->fd_height,
[Kim, Dongwon] I think we could just leave .width/.height setting here
although nothing will go wrong in any cases. Did you have any specific reason
why leaving these uninitialized?
> + .backing_width = m->fd_width,
> + .backing_height = m->fd_height,
> .stride = m->fd_stride,
> .fourcc = m->fd_drm_fourcc,
> .y0_top = m->fd_flags & VIRTIO_GPU_RESOURCE_FLAG_Y_0_TOP,
> --
> 2.41.0.28.gd7d8841f67
