[PATCH xserver 3/4] glamor: Don't fail in glamor_get_formats if not dmabuf_capable.

Mario Kleiner mario.kleiner.de at gmail.com
Mon Apr 30 07:06:09 UTC 2018


If dmabuf_capable is false, because the server "dmabuf_capable"
debug flag isn't set, treat it as successfull query with zero
returned formats, instead of failure.

This allows the servers cache_formats_and_modifiers() function
to cache the fact that formats are not supported during the
current server generation, instead of pointless retesting at
every invocation.

Signed-off-by: Mario Kleiner <mario.kleiner.de at gmail.com>
---
 glamor/glamor_egl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index 9edb509..c38b02e 100644
--- a/glamor/glamor_egl.c
+++ b/glamor/glamor_egl.c
@@ -602,7 +602,7 @@ glamor_get_formats(ScreenPtr screen,
     glamor_egl = glamor_egl_get_screen_private(xf86ScreenToScrn(screen));
 
     if (!glamor_egl->dmabuf_capable)
-        return FALSE;
+        return TRUE;
 
     if (!eglQueryDmaBufFormatsEXT(glamor_egl->display, 0, NULL, &num))
         return FALSE;
-- 
2.7.4



More information about the xorg-devel mailing list