Attachment 140818 Details for Bug 107117 – [PATCH v3] dri3: For 1.2, use root window instead of pixmap drawable

[patch] [PATCH v3] dri3: For 1.2, use root window instead of pixmap drawable

0001-dri3-For-1.2-use-root-window-instead-of-pixmap-drawa.patch (text/plain), 4.07 KB, created by Olivier Fourdan on 2018-07-26 07:49:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Olivier Fourdan

Created: 2018-07-26 07:49:13 UTC

Size: 4.07 KB

patch

obsolete

>From 0dc9ec7e7b4eabdde092f2d8405c95213d0088ea Mon Sep 17 00:00:00 2001
>From: Olivier Fourdan <ofourdan@redhat.com>
>Date: Wed, 25 Jul 2018 15:52:01 +0200
>Subject: [PATCH v3] dri3: For 1.2, use root window instead of pixmap drawable
>
>get_supported_modifiers() and pixmap_from_buffers() requests both
>expect a window as drawable, passing a pixmap will fail as the Xserver
>will fail to match the given drawable to a window.
>
>That leads to dri3_alloc_render_buffer() to return NULL and breaks
>rendering when using GLX_DOUBLEBUFFER on pixmaps.
>
>Query the root window of the pixmap on first init, and use the root
>window instead of the pixmap drawable for get_supported_modifiers()
>and pixmap_from_buffers().
>
>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107117
>Fixes: 069fdd5 ("egl/x11: Support DRI3 v1.1")
>Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
>---
> v2: Use the root window instead of reverting to pixmap_from_buffer()
>     as suggested by Eric.
> v3: Much simpler patch, no need to issue an xcb_get_geometry() twice,
>     we already did it in dri3_update_drawable(), just save the root window
>     from there... Sorry for the noise!
>
> src/loader/loader_dri3_helper.c | 12 +++++++++---
> src/loader/loader_dri3_helper.h |  1 +
> 2 files changed, 10 insertions(+), 3 deletions(-)
>
>diff --git a/src/loader/loader_dri3_helper.c b/src/loader/loader_dri3_helper.c
>index f0ff2f07bd..b8eb87f5aa 100644
>--- a/src/loader/loader_dri3_helper.c
>+++ b/src/loader/loader_dri3_helper.c
>@@ -1149,7 +1149,7 @@ dri3_alloc_render_buffer(struct loader_dri3_drawable *draw, unsigned int format,
>          uint32_t count = 0;
> 
>          mod_cookie = xcb_dri3_get_supported_modifiers(draw->conn,
>-                                                       draw->drawable,
>+                                                       draw->window,
>                                                        depth, buffer->cpp * 8);
>          mod_reply = xcb_dri3_get_supported_modifiers_reply(draw->conn,
>                                                             mod_cookie,
>@@ -1281,7 +1281,7 @@ dri3_alloc_render_buffer(struct loader_dri3_drawable *draw, unsigned int format,
>        buffer->modifier != DRM_FORMAT_MOD_INVALID) {
>       xcb_dri3_pixmap_from_buffers(draw->conn,
>                                    pixmap,
>-                                   draw->drawable,
>+                                   draw->window,
>                                    num_planes,
>                                    width, height,
>                                    buffer->strides[0], buffer->offsets[0],
>@@ -1357,6 +1357,7 @@ dri3_update_drawable(__DRIdrawable *driDrawable,
>       xcb_generic_error_t                       *error;
>       xcb_present_query_capabilities_cookie_t   present_capabilities_cookie;
>       xcb_present_query_capabilities_reply_t    *present_capabilities_reply;
>+      xcb_window_t                               root_win;
> 
>       draw->first_init = false;
> 
>@@ -1394,11 +1395,11 @@ dri3_update_drawable(__DRIdrawable *driDrawable,
>          mtx_unlock(&draw->mtx);
>          return false;
>       }
>-
>       draw->width = geom_reply->width;
>       draw->height = geom_reply->height;
>       draw->depth = geom_reply->depth;
>       draw->vtable->set_drawable_size(draw, draw->width, draw->height);
>+      root_win = geom_reply->root;
> 
>       free(geom_reply);
> 
>@@ -1432,6 +1433,11 @@ dri3_update_drawable(__DRIdrawable *driDrawable,
>          xcb_unregister_for_special_event(draw->conn, draw->special_event);
>          draw->special_event = NULL;
>       }
>+
>+      if (draw->is_pixmap)
>+         draw->window = root_win;
>+      else
>+         draw->window = draw->drawable;
>    }
>    dri3_flush_present_events(draw);
>    mtx_unlock(&draw->mtx);
>diff --git a/src/loader/loader_dri3_helper.h b/src/loader/loader_dri3_helper.h
>index 7e3d82947b..51d000343d 100644
>--- a/src/loader/loader_dri3_helper.h
>+++ b/src/loader/loader_dri3_helper.h
>@@ -114,6 +114,7 @@ struct loader_dri3_drawable {
>    xcb_connection_t *conn;
>    __DRIdrawable *dri_drawable;
>    xcb_drawable_t drawable;
>+   xcb_window_t window;
>    int width;
>    int height;
>    int depth;
>-- 
>2.17.1
>

Actions: View | Splinter Review

Attachments on bug 107117: 140469 | 140540 | 140541 | 140557 | 140735 | 140813 | 140817 | 140818

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.