On Fri, 2008-10-03 at 05:57 +0300, Daniel Stone wrote:
> On Thu, Oct 02, 2008 at 06:53:09PM -0700, Keith Packard wrote:
> > On Fri, 2008-10-03 at 02:01 +0200, Simon Thum wrote:
> > > (b) may suffice. Locking the queue in OsBlockSigs() should do it and fix 
> > > most miEnqueue users.
> > 
> > Or just lock the queue in mieqEnqueue itself; keeping the lock near the
> > code seems like a lot better plan to me.
> 
> Except if the lock is held across the entire event processing, because
> we need to queue events from event processing.

That makes it more important that the mutex cover precisely the values
which will be modified in multiple threads, inserting events into the
queue, and not pulling them out, which is done only in a single thread.

-- 
[EMAIL PROTECTED]

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg mailing list
xorg@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/xorg

Reply via email to