[PATCH xserver 4/5] xwayland: Do not disable glamor if eglstream failed

Olivier Fourdan ofourdan at redhat.com
Thu May 24 14:11:02 UTC 2018


EGL stream requires glamor, but the opposite is not true. So if someone
passes "-eglstream" with a GPU which does not support EGL stream, we
could maybe still try GBM and be lucky.

That allows Wayland compositor to pass "eglstream" regardless of the
actual hardware, if they want to enable EGL stream on GPU which support
it.

Signed-off-by: Olivier Fourdan <ofourdan at redhat.com>
---
 hw/xwayland/xwayland.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/hw/xwayland/xwayland.c b/hw/xwayland/xwayland.c
index cc16edf27..5c61b65dc 100644
--- a/hw/xwayland/xwayland.c
+++ b/hw/xwayland/xwayland.c
@@ -997,11 +997,9 @@ xwl_screen_init(ScreenPtr pScreen, int argc, char **argv)
     if (xwl_screen->glamor) {
 #ifdef XWL_HAS_EGLSTREAM
         if (use_eglstreams) {
-            if (!xwl_glamor_init_eglstream(xwl_screen)) {
-                ErrorF("xwayland glamor: failed to setup eglstream backend, falling back to swaccel\n");
-                xwl_screen->glamor = 0;
-            }
-        } else
+            if (!xwl_glamor_init_eglstream(xwl_screen))
+                ErrorF("xwayland glamor: failed to setup eglstream backend\n");
+        }
 #endif
         if (!xwl_glamor_init_gbm(xwl_screen)) {
             ErrorF("xwayland glamor: failed to setup GBM backend, falling back to sw accel\n");
-- 
2.17.0



More information about the xorg-devel mailing list