Re: [PATCH] GLX/DRI2: Do not expose INTEL_swap_event without swap control

2013-03-18 Thread Keith Packard
Zack Rusin za...@vmware.com writes: Swap events depent on the implementation of ScheduleSwap. By unconditionally enabling GLX_INTEL_swap_event we're breaking the system with drivers that don't support it because the apps are forever stuck waiting for an event that will never be delivered. So

Re: [PATCH] GLX/DRI2: Do not expose INTEL_swap_event without swap control

2013-03-18 Thread Keith Packard
Ian Romanick i...@freedesktop.org writes: Reviewed-by: Ian Romanick ian.d.roman...@intel.com Cc: Keith Packard kei...@keithp.com The last one is important... without it, Keith will never notice your patch. :) I've adjusted my mail filters so I can monitor patches even without having my

Re: [PATCH] GLX/DRI2: Do not expose INTEL_swap_event without swap control

2013-03-13 Thread Ian Romanick
On 03/12/2013 05:10 PM, Zack Rusin wrote: Swap events depent on the implementation of ScheduleSwap. By unconditionally enabling GLX_INTEL_swap_event we're breaking the system with drivers that don't support it because the apps are forever stuck waiting for an event that will never be delivered.

Re: [PATCH] GLX/DRI2: Do not expose INTEL_swap_event without swap control

2013-03-12 Thread Zack Rusin
Swap events depent on the implementation of ScheduleSwap. By unconditionally enabling GLX_INTEL_swap_event we're breaking the system with drivers that don't support it because the apps are forever stuck waiting for an event that will never be delivered. So lets enable the extension only if

[PATCH] GLX/DRI2: Do not expose INTEL_swap_event without swap control

2013-02-14 Thread Zack Rusin
Swap events depent on the implementation of ScheduleSwap. By unconditionally enabling GLX_INTEL_swap_event we're breaking the system with drivers that don't support it because the apps are forever stuck waiting for an event that will never be delivered. So lets enable the extension only if the