On Wed, Aug 20, 2014 at 4:53 AM, Michel Dänzer <[email protected]> wrote: > 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]>
Reviewed-by: Alex Deucher <[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 _______________________________________________ xorg-driver-ati mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-driver-ati
