[PATCH xf86-video-ati] Don't enable DRI3 by default with EXA

Michel Dänzer michel at daenzer.net
Tue Jul 12 08:40:08 UTC 2016


From: Michel Dänzer <michel.daenzer at amd.com>

It doesn't work correctly in all cases, see e.g.
https://bugs.freedesktop.org/show_bug.cgi?id=95475 . I'm not sure this
is fixable, given EXA's architecture.

Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
---
 man/radeon.man   | 5 +++--
 src/radeon_kms.c | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/man/radeon.man b/man/radeon.man
index cacafb5..44603a5 100644
--- a/man/radeon.man
+++ b/man/radeon.man
@@ -271,8 +271,9 @@ Sea Islands.
 Define the maximum level of DRI to enable. Valid values are 2 for DRI2 or 3 for DRI3.
 The default is
 .B 3 for DRI3
-if the driver was compiled for Xorg >= 1.18.3, otherwise
-.B 2 for DRI2.
+if the driver was compiled for Xorg >= 1.18.3 and glamor is enabled, otherwise
+.B 2 for DRI2. Note:
+DRI3 may not work correctly in all cases with EXA, enable at your own risk.
 .TP
 .BI "Option \*qEnablePageFlip\*q \*q" boolean \*q
 Enable DRI2 page flipping.  The default is
diff --git a/src/radeon_kms.c b/src/radeon_kms.c
index d971a39..d21ff2b 100644
--- a/src/radeon_kms.c
+++ b/src/radeon_kms.c
@@ -1788,7 +1788,7 @@ Bool RADEONScreenInit_KMS(SCREEN_INIT_ARGS_DECL)
 #endif
 
 #if XORG_VERSION_CURRENT >= XORG_VERSION_NUMERIC(1,18,3,0,0)
-    value = TRUE;
+    value = info->use_glamor;
 #else
     value = FALSE;
 #endif
-- 
2.8.1



More information about the amd-gfx mailing list