Re: [PATCH xserver 00/10] Re: Refactor egl_backends for wayland registry

2018-06-05 Thread Lyude Paul
Sweet! Everything looks good to me

For the whole series:

Reviewed-by: Lyude Paul 

On Tue, 2018-06-05 at 19:38 +0200, Olivier Fourdan wrote:
> Hi,
> 
> This is a follow-up on https://patchwork.freedesktop.org/series/44095/
> after Lyude and Emil reviews.
> 
> Cheers,
> Olivier
> 
> Olivier Fourdan (10):
>   xwayland: move glamor specific routines
>   xwayland: swap "name" and "id" in init_wl_registry()
>   xwayland: GBM should fail w/out "GL_OES_EGL_image"
>   xwayland: skip drm authentication with render node
>   xwayland: move egl_backend to its own struct
>   xwayland: Add Wayland interfaces check
>   xwayland: move EGL backend init to glamor
>   xwayland: refactor EGL backends for wayland registry
>   xwayland: check for EGLStream backend explicitly
>   xwayland: EGL_IMG_context_priority required by EGLStream
> 
>  hw/xwayland/xwayland-glamor-eglstream.c | 152 +++
>  hw/xwayland/xwayland-glamor-gbm.c   |  69 +++--
>  hw/xwayland/xwayland-glamor.c   | 186 
>  hw/xwayland/xwayland-present.c  |   5 +-
>  hw/xwayland/xwayland.c  |  28 ++--
>  hw/xwayland/xwayland.h  | 151 ++-
>  6 files changed, 369 insertions(+), 222 deletions(-)
> 
-- 
Cheers,
Lyude Paul
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

[PATCH xserver 00/10] Re: Refactor egl_backends for wayland registry

2018-06-05 Thread Olivier Fourdan
Hi,

This is a follow-up on https://patchwork.freedesktop.org/series/44095/
after Lyude and Emil reviews.

Cheers,
Olivier

Olivier Fourdan (10):
  xwayland: move glamor specific routines
  xwayland: swap "name" and "id" in init_wl_registry()
  xwayland: GBM should fail w/out "GL_OES_EGL_image"
  xwayland: skip drm authentication with render node
  xwayland: move egl_backend to its own struct
  xwayland: Add Wayland interfaces check
  xwayland: move EGL backend init to glamor
  xwayland: refactor EGL backends for wayland registry
  xwayland: check for EGLStream backend explicitly
  xwayland: EGL_IMG_context_priority required by EGLStream

 hw/xwayland/xwayland-glamor-eglstream.c | 152 +++
 hw/xwayland/xwayland-glamor-gbm.c   |  69 +++--
 hw/xwayland/xwayland-glamor.c   | 186 
 hw/xwayland/xwayland-present.c  |   5 +-
 hw/xwayland/xwayland.c  |  28 ++--
 hw/xwayland/xwayland.h  | 151 ++-
 6 files changed, 369 insertions(+), 222 deletions(-)

-- 
2.17.1

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel