From: Michel Dänzer <[email protected]>

This reverts commit 4b5060f357a3cb248c9359c92c1e9c42ef6434c8.

Further testing shows that disabling the thread is only a moderate win in
some cases, but a much bigger loss in some other cases.

Signed-off-by: Michel Dänzer <[email protected]>
---
 src/radeon_glamor.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/radeon_glamor.c b/src/radeon_glamor.c
index 02fa658..210ddcf 100644
--- a/src/radeon_glamor.c
+++ b/src/radeon_glamor.c
@@ -132,16 +132,6 @@ radeon_glamor_pre_init(ScrnInfoPtr scrn)
                        "Incompatible glamor version, required >= 0.3.0.\n");
                        return FALSE;
                } else {
-                       /* This prevents the Gallium radeon winsys from spawning
-                        * a separate thread for submitting command streams to
-                        * the kernel. Improves scores of at least gtkperf and
-                        * x11perf -putimage/-shmput/-getimage/-shmget, probably
-                        * because glamor causes too many command stream flushes
-                        * / too much synchronization with pending command
-                        * streams for the separate thread to pay off.
-                        */
-                       setenv("RADEON_THREAD", "0", 0);
-
                        if (glamor_egl_init(scrn, info->dri2.drm_fd)) {
                                xf86DrvMsg(scrn->scrnIndex, X_INFO,
                                           "glamor detected, initialising EGL 
layer.\n");
-- 
2.1.0

_______________________________________________
xorg-driver-ati mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-driver-ati

Reply via email to