xf86-video-ati: Branch 'master'
Michel Dänzer
daenzer at kemper.freedesktop.org
Thu Aug 21 00:58:32 PDT 2014
src/radeon_glamor.c | 10 ----------
1 file changed, 10 deletions(-)
New commits:
commit 7dab6b2fe9cf690b0dae1c127d0374322bd8278e
Author: Michel Dänzer <michel.daenzer at amd.com>
Date: Wed Aug 20 17:51:35 2014 +0900
Revert "glamor: Set environment variable RADEON_THREAD=0"
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.
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
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");
More information about the xorg-commit
mailing list