On Fri, 2008-10-03 at 02:01 +0200, Simon Thum wrote: > It may be constructed, but IMO this means the queue size is not fully > utilizable given head is 'old':
Yes, that's fairly common in queues -- you often can't use the last entry. Not a huge deal if you make the queue big enough. > BTW, given SMP visibility, isn't it a tiny bit risky to have 3 > 'independent' miEventQueue.head reads in miEnqueue? Given the old 'SIGIO' based code, it didn't matter. Even now, it doesn't really matter as in the worst case we fail to store an event when the queue is nearly full but the server is processing events. > (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. -- [EMAIL PROTECTED]
signature.asc
Description: This is a digitally signed message part
_______________________________________________ xorg mailing list xorg@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/xorg
