* [PATCH] virtio-gpu-udmabuf: replacing scanout_width/height with backing_width/height @ 2023-07-05 22:46 Dongwon Kim 2023-07-10 11:57 ` Marc-André Lureau 0 siblings, 1 reply; 5+ messages in thread From: Dongwon Kim @ 2023-07-05 22:46 UTC (permalink / raw) To: qemu-devel Cc: Dongwon Kim, Gerd Hoffmann, Marc-André Lureau, Vivek Kasireddy 'backing_width' and 'backing_height' are commonly used to indicate the size of the whole backing region so it makes sense to use those terms for VGAUDMABuf as well in place of 'scanout_width' and 'scanout_height'. Cc: Gerd Hoffmann <kraxel@redhat.com> Cc: Marc-André Lureau <marcandre.lureau@redhat.com> Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> --- hw/display/virtio-gpu-udmabuf.c | 8 ++++---- include/ui/console.h | 4 ++-- ui/dbus-listener.c | 4 ++-- ui/egl-helpers.c | 4 ++-- ui/gtk-egl.c | 4 ++-- ui/gtk-gl-area.c | 4 ++-- 6 files changed, 14 insertions(+), 14 deletions(-) diff --git a/hw/display/virtio-gpu-udmabuf.c b/hw/display/virtio-gpu-udmabuf.c index ef1a740de5..920d457d4a 100644 --- a/hw/display/virtio-gpu-udmabuf.c +++ b/hw/display/virtio-gpu-udmabuf.c @@ -186,8 +186,8 @@ static VGPUDMABuf dmabuf->buf.stride = fb->stride; dmabuf->buf.x = r->x; dmabuf->buf.y = r->y; - dmabuf->buf.scanout_width = r->width; - dmabuf->buf.scanout_height = r->height; + dmabuf->buf.backing_width = r->width; + dmabuf->buf.backing_height = r->height; dmabuf->buf.fourcc = qemu_pixman_to_drm_format(fb->format); dmabuf->buf.fd = res->dmabuf_fd; dmabuf->buf.allow_fences = true; @@ -218,8 +218,8 @@ int virtio_gpu_update_dmabuf(VirtIOGPU *g, g->dmabuf.primary[scanout_id] = new_primary; qemu_console_resize(scanout->con, - new_primary->buf.scanout_width, - new_primary->buf.scanout_height); + new_primary->buf.backing_width, + new_primary->buf.backing_height); dpy_gl_scanout_dmabuf(scanout->con, &new_primary->buf); if (old_primary) { diff --git a/include/ui/console.h b/include/ui/console.h index f27b2aad4f..3e8b22d6c6 100644 --- a/include/ui/console.h +++ b/include/ui/console.h @@ -201,8 +201,8 @@ typedef struct QemuDmaBuf { uint32_t texture; uint32_t x; uint32_t y; - uint32_t scanout_width; - uint32_t scanout_height; + uint32_t backing_width; + uint32_t backing_height; bool y0_top; void *sync; int fence_fd; diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c index 0240c39510..7d73681cbc 100644 --- a/ui/dbus-listener.c +++ b/ui/dbus-listener.c @@ -420,8 +420,8 @@ static void dbus_scanout_texture(DisplayChangeListener *dcl, .y0_top = backing_y_0_top, .x = x, .y = y, - .scanout_width = w, - .scanout_height = h, + .backing_width = w, + .backing_height = h, }; assert(tex_id); diff --git a/ui/egl-helpers.c b/ui/egl-helpers.c index 8f9fbf583e..6b7be5753d 100644 --- a/ui/egl-helpers.c +++ b/ui/egl-helpers.c @@ -148,8 +148,8 @@ void egl_fb_blit(egl_fb *dst, egl_fb *src, bool flip) if (src->dmabuf) { x1 = src->dmabuf->x; y1 = src->dmabuf->y; - w = src->dmabuf->scanout_width; - h = src->dmabuf->scanout_height; + w = src->dmabuf->backing_width; + h = src->dmabuf->backing_height; } w = (x1 + w) > src->width ? src->width - x1 : w; diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c index d59b8cd7d7..7604696d4a 100644 --- a/ui/gtk-egl.c +++ b/ui/gtk-egl.c @@ -259,8 +259,8 @@ void gd_egl_scanout_dmabuf(DisplayChangeListener *dcl, gd_egl_scanout_texture(dcl, dmabuf->texture, dmabuf->y0_top, dmabuf->width, dmabuf->height, - dmabuf->x, dmabuf->y, dmabuf->scanout_width, - dmabuf->scanout_height, NULL); + dmabuf->x, dmabuf->y, dmabuf->backing_width, + dmabuf->backing_height, NULL); if (dmabuf->allow_fences) { vc->gfx.guest_fb.dmabuf = dmabuf; diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c index 7367dfd793..3337a4baa3 100644 --- a/ui/gtk-gl-area.c +++ b/ui/gtk-gl-area.c @@ -300,8 +300,8 @@ void gd_gl_area_scanout_dmabuf(DisplayChangeListener *dcl, gd_gl_area_scanout_texture(dcl, dmabuf->texture, dmabuf->y0_top, dmabuf->width, dmabuf->height, - dmabuf->x, dmabuf->y, dmabuf->scanout_width, - dmabuf->scanout_height, NULL); + dmabuf->x, dmabuf->y, dmabuf->backing_width, + dmabuf->backing_height, NULL); if (dmabuf->allow_fences) { vc->gfx.guest_fb.dmabuf = dmabuf; -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] virtio-gpu-udmabuf: replacing scanout_width/height with backing_width/height 2023-07-05 22:46 [PATCH] virtio-gpu-udmabuf: replacing scanout_width/height with backing_width/height Dongwon Kim @ 2023-07-10 11:57 ` Marc-André Lureau 2023-07-13 2:08 ` Kim, Dongwon 2023-07-13 4:04 ` [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties Dongwon Kim 0 siblings, 2 replies; 5+ messages in thread From: Marc-André Lureau @ 2023-07-10 11:57 UTC (permalink / raw) To: Dongwon Kim; +Cc: qemu-devel, Gerd Hoffmann, Vivek Kasireddy [-- Attachment #1: Type: text/plain, Size: 5339 bytes --] Hi On Thu, Jul 6, 2023 at 3:10 AM Dongwon Kim <dongwon.kim@intel.com> wrote: > 'backing_width' and 'backing_height' are commonly used to indicate the size > of the whole backing region so it makes sense to use those terms for > VGAUDMABuf as well in place of 'scanout_width' and 'scanout_height'. > > Cc: Gerd Hoffmann <kraxel@redhat.com> > Cc: Marc-André Lureau <marcandre.lureau@redhat.com> > Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> > Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> > --- > hw/display/virtio-gpu-udmabuf.c | 8 ++++---- > include/ui/console.h | 4 ++-- > ui/dbus-listener.c | 4 ++-- > ui/egl-helpers.c | 4 ++-- > ui/gtk-egl.c | 4 ++-- > ui/gtk-gl-area.c | 4 ++-- > 6 files changed, 14 insertions(+), 14 deletions(-) > > diff --git a/hw/display/virtio-gpu-udmabuf.c > b/hw/display/virtio-gpu-udmabuf.c > index ef1a740de5..920d457d4a 100644 > --- a/hw/display/virtio-gpu-udmabuf.c > +++ b/hw/display/virtio-gpu-udmabuf.c > @@ -186,8 +186,8 @@ static VGPUDMABuf > dmabuf->buf.stride = fb->stride; > dmabuf->buf.x = r->x; > dmabuf->buf.y = r->y; > - dmabuf->buf.scanout_width = r->width; > - dmabuf->buf.scanout_height = r->height; > + dmabuf->buf.backing_width = r->width; > + dmabuf->buf.backing_height = r->height; > dmabuf->buf.fourcc = qemu_pixman_to_drm_format(fb->format); > dmabuf->buf.fd = res->dmabuf_fd; > dmabuf->buf.allow_fences = true; > @@ -218,8 +218,8 @@ int virtio_gpu_update_dmabuf(VirtIOGPU *g, > > g->dmabuf.primary[scanout_id] = new_primary; > qemu_console_resize(scanout->con, > - new_primary->buf.scanout_width, > - new_primary->buf.scanout_height); > + new_primary->buf.backing_width, > + new_primary->buf.backing_height); > dpy_gl_scanout_dmabuf(scanout->con, &new_primary->buf); > > if (old_primary) { > diff --git a/include/ui/console.h b/include/ui/console.h > index f27b2aad4f..3e8b22d6c6 100644 > --- a/include/ui/console.h > +++ b/include/ui/console.h > @@ -201,8 +201,8 @@ typedef struct QemuDmaBuf { > uint32_t texture; > uint32_t x; > uint32_t y; > - uint32_t scanout_width; > - uint32_t scanout_height; > + uint32_t backing_width; > + uint32_t backing_height; > bool y0_top; > void *sync; > int fence_fd; > diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c > index 0240c39510..7d73681cbc 100644 > --- a/ui/dbus-listener.c > +++ b/ui/dbus-listener.c > @@ -420,8 +420,8 @@ static void dbus_scanout_texture(DisplayChangeListener > *dcl, > .y0_top = backing_y_0_top, > .x = x, > .y = y, > - .scanout_width = w, > - .scanout_height = h, > + .backing_width = w, > + .backing_height = h, > This is not consistent with the function arguments. I think it should be after: .width = w, .height = h, .backing_width = backing_wdth, .backing_height = backing_height Hopefully this inconsistency is not repeated elsewhere. thanks > }; > > assert(tex_id); > diff --git a/ui/egl-helpers.c b/ui/egl-helpers.c > index 8f9fbf583e..6b7be5753d 100644 > --- a/ui/egl-helpers.c > +++ b/ui/egl-helpers.c > @@ -148,8 +148,8 @@ void egl_fb_blit(egl_fb *dst, egl_fb *src, bool flip) > if (src->dmabuf) { > x1 = src->dmabuf->x; > y1 = src->dmabuf->y; > - w = src->dmabuf->scanout_width; > - h = src->dmabuf->scanout_height; > + w = src->dmabuf->backing_width; > + h = src->dmabuf->backing_height; > } > > w = (x1 + w) > src->width ? src->width - x1 : w; > diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c > index d59b8cd7d7..7604696d4a 100644 > --- a/ui/gtk-egl.c > +++ b/ui/gtk-egl.c > @@ -259,8 +259,8 @@ void gd_egl_scanout_dmabuf(DisplayChangeListener *dcl, > > gd_egl_scanout_texture(dcl, dmabuf->texture, > dmabuf->y0_top, dmabuf->width, dmabuf->height, > - dmabuf->x, dmabuf->y, dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->x, dmabuf->y, dmabuf->backing_width, > + dmabuf->backing_height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c > index 7367dfd793..3337a4baa3 100644 > --- a/ui/gtk-gl-area.c > +++ b/ui/gtk-gl-area.c > @@ -300,8 +300,8 @@ void gd_gl_area_scanout_dmabuf(DisplayChangeListener > *dcl, > > gd_gl_area_scanout_texture(dcl, dmabuf->texture, > dmabuf->y0_top, dmabuf->width, > dmabuf->height, > - dmabuf->x, dmabuf->y, > dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->x, dmabuf->y, > dmabuf->backing_width, > + dmabuf->backing_height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > -- > 2.34.1 > > > -- Marc-André Lureau [-- Attachment #2: Type: text/html, Size: 7351 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] virtio-gpu-udmabuf: replacing scanout_width/height with backing_width/height 2023-07-10 11:57 ` Marc-André Lureau @ 2023-07-13 2:08 ` Kim, Dongwon 2023-07-13 4:04 ` [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties Dongwon Kim 1 sibling, 0 replies; 5+ messages in thread From: Kim, Dongwon @ 2023-07-13 2:08 UTC (permalink / raw) To: Marc-André Lureau; +Cc: qemu-devel, Gerd Hoffmann, Vivek Kasireddy On 7/10/2023 4:57 AM, Marc-André Lureau wrote: > Hi > > On Thu, Jul 6, 2023 at 3:10 AM Dongwon Kim <dongwon.kim@intel.com> wrote: > > 'backing_width' and 'backing_height' are commonly used to indicate > the size > of the whole backing region so it makes sense to use those terms for > VGAUDMABuf as well in place of 'scanout_width' and 'scanout_height'. > > Cc: Gerd Hoffmann <kraxel@redhat.com> > Cc: Marc-André Lureau <marcandre.lureau@redhat.com> > Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> > Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> > --- > hw/display/virtio-gpu-udmabuf.c | 8 ++++---- > include/ui/console.h | 4 ++-- > ui/dbus-listener.c | 4 ++-- > ui/egl-helpers.c | 4 ++-- > ui/gtk-egl.c | 4 ++-- > ui/gtk-gl-area.c | 4 ++-- > 6 files changed, 14 insertions(+), 14 deletions(-) > > diff --git a/hw/display/virtio-gpu-udmabuf.c > b/hw/display/virtio-gpu-udmabuf.c > index ef1a740de5..920d457d4a 100644 > --- a/hw/display/virtio-gpu-udmabuf.c > +++ b/hw/display/virtio-gpu-udmabuf.c > @@ -186,8 +186,8 @@ static VGPUDMABuf > dmabuf->buf.stride = fb->stride; > dmabuf->buf.x = r->x; > dmabuf->buf.y = r->y; > - dmabuf->buf.scanout_width = r->width; > - dmabuf->buf.scanout_height = r->height; > + dmabuf->buf.backing_width = r->width; > + dmabuf->buf.backing_height = r->height; > dmabuf->buf.fourcc = qemu_pixman_to_drm_format(fb->format); > dmabuf->buf.fd = res->dmabuf_fd; > dmabuf->buf.allow_fences = true; > @@ -218,8 +218,8 @@ int virtio_gpu_update_dmabuf(VirtIOGPU *g, > > g->dmabuf.primary[scanout_id] = new_primary; > qemu_console_resize(scanout->con, > - new_primary->buf.scanout_width, > - new_primary->buf.scanout_height); > + new_primary->buf.backing_width, > + new_primary->buf.backing_height); > dpy_gl_scanout_dmabuf(scanout->con, &new_primary->buf); > > if (old_primary) { > diff --git a/include/ui/console.h b/include/ui/console.h > index f27b2aad4f..3e8b22d6c6 100644 > --- a/include/ui/console.h > +++ b/include/ui/console.h > @@ -201,8 +201,8 @@ typedef struct QemuDmaBuf { > uint32_t texture; > uint32_t x; > uint32_t y; > - uint32_t scanout_width; > - uint32_t scanout_height; > + uint32_t backing_width; > + uint32_t backing_height; > bool y0_top; > void *sync; > int fence_fd; > diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c > index 0240c39510..7d73681cbc 100644 > --- a/ui/dbus-listener.c > +++ b/ui/dbus-listener.c > @@ -420,8 +420,8 @@ static void > dbus_scanout_texture(DisplayChangeListener *dcl, > .y0_top = backing_y_0_top, > .x = x, > .y = y, > - .scanout_width = w, > - .scanout_height = h, > + .backing_width = w, > + .backing_height = h, > > > This is not consistent with the function arguments. I think it should > be after: > > .width = w, .height = h, .backing_width = backing_wdth, > .backing_height = backing_height > > Hopefully this inconsistency is not repeated elsewhere. > Yes, you are right. Backing_* is for the whole surface. And normal width/height or w/h specifies the sub region as you mentioned earlier in all other places. Inconsistency was caused in QemuDmabuf where width/height was used as backing_width/height. We should have corrected it first. I will send another version of patch to correct this. > thanks > > }; > > assert(tex_id); > diff --git a/ui/egl-helpers.c b/ui/egl-helpers.c > index 8f9fbf583e..6b7be5753d 100644 > --- a/ui/egl-helpers.c > +++ b/ui/egl-helpers.c > @@ -148,8 +148,8 @@ void egl_fb_blit(egl_fb *dst, egl_fb *src, > bool flip) > if (src->dmabuf) { > x1 = src->dmabuf->x; > y1 = src->dmabuf->y; > - w = src->dmabuf->scanout_width; > - h = src->dmabuf->scanout_height; > + w = src->dmabuf->backing_width; > + h = src->dmabuf->backing_height; > } > > w = (x1 + w) > src->width ? src->width - x1 : w; > diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c > index d59b8cd7d7..7604696d4a 100644 > --- a/ui/gtk-egl.c > +++ b/ui/gtk-egl.c > @@ -259,8 +259,8 @@ void > gd_egl_scanout_dmabuf(DisplayChangeListener *dcl, > > gd_egl_scanout_texture(dcl, dmabuf->texture, > dmabuf->y0_top, dmabuf->width, > dmabuf->height, > - dmabuf->x, dmabuf->y, > dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->x, dmabuf->y, > dmabuf->backing_width, > + dmabuf->backing_height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c > index 7367dfd793..3337a4baa3 100644 > --- a/ui/gtk-gl-area.c > +++ b/ui/gtk-gl-area.c > @@ -300,8 +300,8 @@ void > gd_gl_area_scanout_dmabuf(DisplayChangeListener *dcl, > > gd_gl_area_scanout_texture(dcl, dmabuf->texture, > dmabuf->y0_top, dmabuf->width, > dmabuf->height, > - dmabuf->x, dmabuf->y, > dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->x, dmabuf->y, > dmabuf->backing_width, > + dmabuf->backing_height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > -- > 2.34.1 > > > > > -- > Marc-André Lureau ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties 2023-07-10 11:57 ` Marc-André Lureau 2023-07-13 2:08 ` Kim, Dongwon @ 2023-07-13 4:04 ` Dongwon Kim 2023-07-17 9:25 ` Marc-André Lureau 1 sibling, 1 reply; 5+ messages in thread From: Dongwon Kim @ 2023-07-13 4:04 UTC (permalink / raw) To: qemu-devel Cc: Dongwon Kim, Gerd Hoffmann, Marc-André Lureau, Vivek Kasireddy Replace 'width' and 'height' in QemuDmaBuf with 'backing_widht' and 'backing_height' as these commonly indicate the size of the whole surface (e.g. guest's Xorg extended display). Then use 'width' and 'height' for sub region in there (e.g. guest's scanouts). Cc: Gerd Hoffmann <kraxel@redhat.com> Cc: Marc-André Lureau <marcandre.lureau@redhat.com> Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> --- hw/display/virtio-gpu-udmabuf.c | 12 ++++++------ include/ui/console.h | 4 ++-- ui/dbus-listener.c | 8 ++++---- ui/egl-helpers.c | 8 ++++---- ui/gtk-egl.c | 8 ++++---- ui/gtk-gl-area.c | 6 +++--- 6 files changed, 23 insertions(+), 23 deletions(-) diff --git a/hw/display/virtio-gpu-udmabuf.c b/hw/display/virtio-gpu-udmabuf.c index ef1a740de5..047758582c 100644 --- a/hw/display/virtio-gpu-udmabuf.c +++ b/hw/display/virtio-gpu-udmabuf.c @@ -181,13 +181,13 @@ static VGPUDMABuf } dmabuf = g_new0(VGPUDMABuf, 1); - dmabuf->buf.width = fb->width; - dmabuf->buf.height = fb->height; + dmabuf->buf.width = r->width; + dmabuf->buf.height = r->height; dmabuf->buf.stride = fb->stride; dmabuf->buf.x = r->x; dmabuf->buf.y = r->y; - dmabuf->buf.scanout_width = r->width; - dmabuf->buf.scanout_height = r->height; + dmabuf->buf.backing_width = fb->width; + dmabuf->buf.backing_height = fb->height;; dmabuf->buf.fourcc = qemu_pixman_to_drm_format(fb->format); dmabuf->buf.fd = res->dmabuf_fd; dmabuf->buf.allow_fences = true; @@ -218,8 +218,8 @@ int virtio_gpu_update_dmabuf(VirtIOGPU *g, g->dmabuf.primary[scanout_id] = new_primary; qemu_console_resize(scanout->con, - new_primary->buf.scanout_width, - new_primary->buf.scanout_height); + new_primary->buf.width, + new_primary->buf.height); dpy_gl_scanout_dmabuf(scanout->con, &new_primary->buf); if (old_primary) { diff --git a/include/ui/console.h b/include/ui/console.h index f27b2aad4f..3e8b22d6c6 100644 --- a/include/ui/console.h +++ b/include/ui/console.h @@ -201,8 +201,8 @@ typedef struct QemuDmaBuf { uint32_t texture; uint32_t x; uint32_t y; - uint32_t scanout_width; - uint32_t scanout_height; + uint32_t backing_width; + uint32_t backing_height; bool y0_top; void *sync; int fence_fd; diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c index 0240c39510..68ff343799 100644 --- a/ui/dbus-listener.c +++ b/ui/dbus-listener.c @@ -415,13 +415,13 @@ static void dbus_scanout_texture(DisplayChangeListener *dcl, backing_width, backing_height, x, y, w, h); #ifdef CONFIG_GBM QemuDmaBuf dmabuf = { - .width = backing_width, - .height = backing_height, + .width = w, + .height = h, .y0_top = backing_y_0_top, .x = x, .y = y, - .scanout_width = w, - .scanout_height = h, + .backing_width = backing_width, + .backing_height = backing_height, }; assert(tex_id); diff --git a/ui/egl-helpers.c b/ui/egl-helpers.c index 8f9fbf583e..bc0960a9ec 100644 --- a/ui/egl-helpers.c +++ b/ui/egl-helpers.c @@ -148,8 +148,8 @@ void egl_fb_blit(egl_fb *dst, egl_fb *src, bool flip) if (src->dmabuf) { x1 = src->dmabuf->x; y1 = src->dmabuf->y; - w = src->dmabuf->scanout_width; - h = src->dmabuf->scanout_height; + w = src->dmabuf->width; + h = src->dmabuf->height; } w = (x1 + w) > src->width ? src->width - x1 : w; @@ -314,9 +314,9 @@ void egl_dmabuf_import_texture(QemuDmaBuf *dmabuf) } attrs[i++] = EGL_WIDTH; - attrs[i++] = dmabuf->width; + attrs[i++] = dmabuf->backing_width; attrs[i++] = EGL_HEIGHT; - attrs[i++] = dmabuf->height; + attrs[i++] = dmabuf->backing_height;; attrs[i++] = EGL_LINUX_DRM_FOURCC_EXT; attrs[i++] = dmabuf->fourcc; diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c index d59b8cd7d7..a37ad6c9db 100644 --- a/ui/gtk-egl.c +++ b/ui/gtk-egl.c @@ -258,9 +258,9 @@ void gd_egl_scanout_dmabuf(DisplayChangeListener *dcl, } gd_egl_scanout_texture(dcl, dmabuf->texture, - dmabuf->y0_top, dmabuf->width, dmabuf->height, - dmabuf->x, dmabuf->y, dmabuf->scanout_width, - dmabuf->scanout_height, NULL); + dmabuf->y0_top, dmabuf->backing_width, dmabuf->backing_height, + dmabuf->x, dmabuf->y, dmabuf->width, + dmabuf->height, NULL); if (dmabuf->allow_fences) { vc->gfx.guest_fb.dmabuf = dmabuf; @@ -280,7 +280,7 @@ void gd_egl_cursor_dmabuf(DisplayChangeListener *dcl, if (!dmabuf->texture) { return; } - egl_fb_setup_for_tex(&vc->gfx.cursor_fb, dmabuf->width, dmabuf->height, + egl_fb_setup_for_tex(&vc->gfx.cursor_fb, dmabuf->backing_width, dmabuf->backing_height, dmabuf->texture, false); } else { egl_fb_destroy(&vc->gfx.cursor_fb); diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c index 7367dfd793..ea3d50598e 100644 --- a/ui/gtk-gl-area.c +++ b/ui/gtk-gl-area.c @@ -299,9 +299,9 @@ void gd_gl_area_scanout_dmabuf(DisplayChangeListener *dcl, } gd_gl_area_scanout_texture(dcl, dmabuf->texture, - dmabuf->y0_top, dmabuf->width, dmabuf->height, - dmabuf->x, dmabuf->y, dmabuf->scanout_width, - dmabuf->scanout_height, NULL); + dmabuf->y0_top, dmabuf->backing_width, dmabuf->backing_height, + dmabuf->x, dmabuf->y, dmabuf->width, + dmabuf->height, NULL); if (dmabuf->allow_fences) { vc->gfx.guest_fb.dmabuf = dmabuf; -- 2.20.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties 2023-07-13 4:04 ` [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties Dongwon Kim @ 2023-07-17 9:25 ` Marc-André Lureau 0 siblings, 0 replies; 5+ messages in thread From: Marc-André Lureau @ 2023-07-17 9:25 UTC (permalink / raw) To: Dongwon Kim; +Cc: qemu-devel, Gerd Hoffmann, Vivek Kasireddy [-- Attachment #1: Type: text/plain, Size: 6742 bytes --] On Thu, Jul 13, 2023 at 8:28 AM Dongwon Kim <dongwon.kim@intel.com> wrote: > Replace 'width' and 'height' in QemuDmaBuf with 'backing_widht' > and 'backing_height' as these commonly indicate the size of the > whole surface (e.g. guest's Xorg extended display). Then use > 'width' and 'height' for sub region in there (e.g. guest's > scanouts). > > Cc: Gerd Hoffmann <kraxel@redhat.com> > Cc: Marc-André Lureau <marcandre.lureau@redhat.com> > Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> > Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> > Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> > --- > hw/display/virtio-gpu-udmabuf.c | 12 ++++++------ > include/ui/console.h | 4 ++-- > ui/dbus-listener.c | 8 ++++---- > ui/egl-helpers.c | 8 ++++---- > ui/gtk-egl.c | 8 ++++---- > ui/gtk-gl-area.c | 6 +++--- > 6 files changed, 23 insertions(+), 23 deletions(-) > > diff --git a/hw/display/virtio-gpu-udmabuf.c > b/hw/display/virtio-gpu-udmabuf.c > index ef1a740de5..047758582c 100644 > --- a/hw/display/virtio-gpu-udmabuf.c > +++ b/hw/display/virtio-gpu-udmabuf.c > @@ -181,13 +181,13 @@ static VGPUDMABuf > } > > dmabuf = g_new0(VGPUDMABuf, 1); > - dmabuf->buf.width = fb->width; > - dmabuf->buf.height = fb->height; > + dmabuf->buf.width = r->width; > + dmabuf->buf.height = r->height; > dmabuf->buf.stride = fb->stride; > dmabuf->buf.x = r->x; > dmabuf->buf.y = r->y; > - dmabuf->buf.scanout_width = r->width; > - dmabuf->buf.scanout_height = r->height; > + dmabuf->buf.backing_width = fb->width; > + dmabuf->buf.backing_height = fb->height;; > dmabuf->buf.fourcc = qemu_pixman_to_drm_format(fb->format); > dmabuf->buf.fd = res->dmabuf_fd; > dmabuf->buf.allow_fences = true; > @@ -218,8 +218,8 @@ int virtio_gpu_update_dmabuf(VirtIOGPU *g, > > g->dmabuf.primary[scanout_id] = new_primary; > qemu_console_resize(scanout->con, > - new_primary->buf.scanout_width, > - new_primary->buf.scanout_height); > + new_primary->buf.width, > + new_primary->buf.height); > dpy_gl_scanout_dmabuf(scanout->con, &new_primary->buf); > > if (old_primary) { > diff --git a/include/ui/console.h b/include/ui/console.h > index f27b2aad4f..3e8b22d6c6 100644 > --- a/include/ui/console.h > +++ b/include/ui/console.h > @@ -201,8 +201,8 @@ typedef struct QemuDmaBuf { > uint32_t texture; > uint32_t x; > uint32_t y; > - uint32_t scanout_width; > - uint32_t scanout_height; > + uint32_t backing_width; > + uint32_t backing_height; > bool y0_top; > void *sync; > int fence_fd; > diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c > index 0240c39510..68ff343799 100644 > --- a/ui/dbus-listener.c > +++ b/ui/dbus-listener.c > @@ -415,13 +415,13 @@ static void > dbus_scanout_texture(DisplayChangeListener *dcl, > backing_width, backing_height, x, y, w, h); > #ifdef CONFIG_GBM > QemuDmaBuf dmabuf = { > - .width = backing_width, > - .height = backing_height, > + .width = w, > + .height = h, > .y0_top = backing_y_0_top, > .x = x, > .y = y, > - .scanout_width = w, > - .scanout_height = h, > + .backing_width = backing_width, > + .backing_height = backing_height, > }; > > assert(tex_id); > diff --git a/ui/egl-helpers.c b/ui/egl-helpers.c > index 8f9fbf583e..bc0960a9ec 100644 > --- a/ui/egl-helpers.c > +++ b/ui/egl-helpers.c > @@ -148,8 +148,8 @@ void egl_fb_blit(egl_fb *dst, egl_fb *src, bool flip) > if (src->dmabuf) { > x1 = src->dmabuf->x; > y1 = src->dmabuf->y; > - w = src->dmabuf->scanout_width; > - h = src->dmabuf->scanout_height; > + w = src->dmabuf->width; > + h = src->dmabuf->height; > } > > w = (x1 + w) > src->width ? src->width - x1 : w; > @@ -314,9 +314,9 @@ void egl_dmabuf_import_texture(QemuDmaBuf *dmabuf) > } > > attrs[i++] = EGL_WIDTH; > - attrs[i++] = dmabuf->width; > + attrs[i++] = dmabuf->backing_width; > attrs[i++] = EGL_HEIGHT; > - attrs[i++] = dmabuf->height; > + attrs[i++] = dmabuf->backing_height;; > attrs[i++] = EGL_LINUX_DRM_FOURCC_EXT; > attrs[i++] = dmabuf->fourcc; > > diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c > index d59b8cd7d7..a37ad6c9db 100644 > --- a/ui/gtk-egl.c > +++ b/ui/gtk-egl.c > @@ -258,9 +258,9 @@ void gd_egl_scanout_dmabuf(DisplayChangeListener *dcl, > } > > gd_egl_scanout_texture(dcl, dmabuf->texture, > - dmabuf->y0_top, dmabuf->width, dmabuf->height, > - dmabuf->x, dmabuf->y, dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->y0_top, dmabuf->backing_width, > dmabuf->backing_height, > + dmabuf->x, dmabuf->y, dmabuf->width, > + dmabuf->height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > @@ -280,7 +280,7 @@ void gd_egl_cursor_dmabuf(DisplayChangeListener *dcl, > if (!dmabuf->texture) { > return; > } > - egl_fb_setup_for_tex(&vc->gfx.cursor_fb, dmabuf->width, > dmabuf->height, > + egl_fb_setup_for_tex(&vc->gfx.cursor_fb, dmabuf->backing_width, > dmabuf->backing_height, > dmabuf->texture, false); > } else { > egl_fb_destroy(&vc->gfx.cursor_fb); > diff --git a/ui/gtk-gl-area.c b/ui/gtk-gl-area.c > index 7367dfd793..ea3d50598e 100644 > --- a/ui/gtk-gl-area.c > +++ b/ui/gtk-gl-area.c > @@ -299,9 +299,9 @@ void gd_gl_area_scanout_dmabuf(DisplayChangeListener > *dcl, > } > > gd_gl_area_scanout_texture(dcl, dmabuf->texture, > - dmabuf->y0_top, dmabuf->width, > dmabuf->height, > - dmabuf->x, dmabuf->y, > dmabuf->scanout_width, > - dmabuf->scanout_height, NULL); > + dmabuf->y0_top, dmabuf->backing_width, > dmabuf->backing_height, > + dmabuf->x, dmabuf->y, dmabuf->width, > + dmabuf->height, NULL); > > if (dmabuf->allow_fences) { > vc->gfx.guest_fb.dmabuf = dmabuf; > -- > 2.20.1 > > > -- Marc-André Lureau [-- Attachment #2: Type: text/html, Size: 8908 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-07-17 9:25 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-05 22:46 [PATCH] virtio-gpu-udmabuf: replacing scanout_width/height with backing_width/height Dongwon Kim 2023-07-10 11:57 ` Marc-André Lureau 2023-07-13 2:08 ` Kim, Dongwon 2023-07-13 4:04 ` [PATCH] virtio-gpu-udmabuf: correct naming of QemuDmaBuf size properties Dongwon Kim 2023-07-17 9:25 ` Marc-André Lureau
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).