[Mesa-dev] [PATCH 1/3] egl/x11: Honor the EGL_PLATFORM_X11_SCREEN_EXT attribute

[Emil Velikov]

On 3 May 2017 at 16:57, Adam Jackson <ajax at redhat.com> wrote:
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  src/egl/drivers/dri2/platform_x11.c |  2 +-
>  src/egl/main/egldisplay.c           | 19 ++++++++++++-------
>  src/egl/main/egldisplay.h           |  1 +
>  3 files changed, 14 insertions(+), 8 deletions(-)
>
> diff --git a/src/egl/drivers/dri2/platform_x11.c b/src/egl/drivers/dri2/platform_x11.c
> index 2f1086e28f..c78656a5be 100644
> --- a/src/egl/drivers/dri2/platform_x11.c
> +++ b/src/egl/drivers/dri2/platform_x11.c
> @@ -1191,7 +1191,7 @@ dri2_get_xcb_connection(_EGLDriver *drv, _EGLDisplay *disp,
>                          struct dri2_egl_display *dri2_dpy)
>  {
>     xcb_screen_iterator_t s;
> -   int screen = 0;
> +   int screen = (uintptr_t)disp->Options.Platform;
>     const char *msg;
>
Seems like we don't propagate the screen to the DRI module. See the
createNewScreen* calls in dri2_create_screen().
Not a huge deal breaker, but can we fix that?

Pardon for missing it last round :-\

Thanks
Emil