[PATCH] radeon: proper DRI2 pending events handling when client gone. (v3)

[Michel Dänzer]

On Sam, 2010-09-18 at 12:03 +0200, Christopher James Halse Rogers
wrote: 
> On Tue, 2010-09-14 at 22:55 +0200, Oldřich Jedlička wrote:
> > Properly handle asynchronous DRI2 events for disconnected clients.
> > Track client's pending requests and mark them as invalid when the
> > client disconnects.
> > 
> > This is based on the version from Alban Browaeys in bug #29065.
> > 
> > v1 (Alban Browaeys): Based upon a detailed explanation from Oldřich
> > Jedlička and comments from Christopher James Halse Rogers.
> > on http://lists.x.org/archives/xorg-driver-ati/2010-August/016780.html .
> > 
> > v2: Updated version to apply on master. Removed unnecessary
> > client_index field from _DRI2FrameEvent. Added freeing/removing from
> > list to failed paths of radeon_dri2_schedule_wait_msc and
> > radeon_dri2_schedule_swap.
> > 
> > v3: Adopt to older xorg-server that doesn't have dixRegisterPrivateKey.
> > ---
> >  src/radeon_dri2.c |  149 +++++++++++++++++++++++++++++++++++++++++++++--------
> >  1 files changed, 128 insertions(+), 21 deletions(-)
> > 
> > diff --git a/src/radeon_dri2.c b/src/radeon_dri2.c
> > index ed7fdd6..054b405 100644
> > --- a/src/radeon_dri2.c
> > +++ b/src/radeon_dri2.c
> > @@ -37,6 +37,7 @@
> >  #include "radeon.h"
> >  #include "radeon_dri2.h"
> >  #include "radeon_version.h"
> > +#include "list.h"
> 
> This header was only introduced in xserver 1.8.  I presume that it's
> expected that radeon can build against older X servers, so you might
> need to conditionally copy this into the DDX.

That shouldn't be necessary (for this), as the whole DRI2 swap
scheduling support was also only added in 1.8.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer