[PATCH 1/2] OS support: fix writeable client vs IgnoreClient behavior

Jesse Barnes jbarnes at virtuousgeek.org
Mon Jun 28 19:14:41 PDT 2010


On Mon, 28 Jun 2010 18:48:51 -0700
Keith Packard <keithp at keithp.com> wrote:

> On Mon, 28 Jun 2010 17:43:28 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> 
> > This is really a question of which behavior we want to preserve:
> >   1) clearing of IgnoredClientsWithInput across multiple IgnoreClient
> >   calls
> > or
> >   2) allowing multiple IgnoreClient and AttendClient calls to have any
> >   effect at all except when first ignoring and finally attending
> 
> I think that 2) is what we want -- permitting nesting of ignore/attend
> calls with the client blocked until all of the reasons for suspending
> the client are cleared.
> 
> Can anyone think of a reason to want the other behaviour in
> non-cooperating portions of the server?
> 
> > Here's what the counter patch looks like.  It needs to be matched with
> > a change to my DRI2 patch to unconditionally call AttendClient so the
> > counts match up.
> 
> This looks good to me. I assume the client is allocated with calloc so
> that the count starts out at zero?

Initially I just checked the alloc, and no it doesn't use calloc.  But
it does memset it to 0 in InitClient, so my initialization to 0 is
unnecessary.

> Reviewed-by: Keith Packard <keithp at keithp.com>
 
Thanks.  Do you want to apply it with that change or shall I send an
update?

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the xorg-devel mailing list