Re: [Mesa-dev] [PATCH 1/3] glx: remove glFlush call in glXSwapBuffers

2012-11-08 Thread Marek Olšák
On Thu, Nov 8, 2012 at 7:59 PM, Ian Romanick wrote: > On 11/08/2012 05:48 AM, Marek Olšák wrote: >> >> Flushing here is unnecessary. The drivers which always flush in >> dri2Throttle >> or __DRI2flushExtensionRec::flush are just wasting time here. >> >> st/mesa is updated to do what glFlush does.

Re: [Mesa-dev] [PATCH 1/3] glx: remove glFlush call in glXSwapBuffers

2012-11-08 Thread Ian Romanick
On 11/08/2012 05:48 AM, Marek Olšák wrote: Flushing here is unnecessary. The drivers which always flush in dri2Throttle or __DRI2flushExtensionRec::flush are just wasting time here. st/mesa is updated to do what glFlush does. XXX I guess some other drivers should be updated to flush by themselv

Re: [Mesa-dev] [PATCH 1/3] glx: remove glFlush call in glXSwapBuffers

2012-11-08 Thread Marek Olšák
On Thu, Nov 8, 2012 at 3:04 PM, Michel Dänzer wrote: > On Don, 2012-11-08 at 14:48 +0100, Marek Olšák wrote: >> Flushing here is unnecessary. The drivers which always flush in dri2Throttle >> or __DRI2flushExtensionRec::flush are just wasting time here. >> >> st/mesa is updated to do what glFlush

Re: [Mesa-dev] [PATCH 1/3] glx: remove glFlush call in glXSwapBuffers

2012-11-08 Thread Michel Dänzer
On Don, 2012-11-08 at 14:48 +0100, Marek Olšák wrote: > Flushing here is unnecessary. The drivers which always flush in dri2Throttle > or __DRI2flushExtensionRec::flush are just wasting time here. > > st/mesa is updated to do what glFlush does. > > XXX I guess some other drivers should be update

[Mesa-dev] [PATCH 1/3] glx: remove glFlush call in glXSwapBuffers

2012-11-08 Thread Marek Olšák
Flushing here is unnecessary. The drivers which always flush in dri2Throttle or __DRI2flushExtensionRec::flush are just wasting time here. st/mesa is updated to do what glFlush does. XXX I guess some other drivers should be updated to flush by themselves if they don't? --- src/glx/glxcmds.c