On Oct 16, 2011, at 10:12 PM, Keith Packard wrote: > On Sun, 16 Oct 2011 21:16:13 -0700, Jeremy Huddleston <[email protected]> > wrote: >> >> The last 64 events in the event queue will be reserved for release >> events in order to help return the system to a cleaner state when >> it comes back from a soft wedge. > > Yeah, seems reasonable to me. > >> /* Queue size must be a power of 2 */ >> -#define QUEUE_INITIAL_SIZE 128 >> +#define QUEUE_INITIAL_SIZE 256 > > Just noticed this -- I don't see any place which requires a power of two > initial size.
Yeah, that's probably true actually. It just doubles in size, but there's nothing that actually requires it to be a power of 2. The only thing that is required for this to behave right is that max == initial * 2^i for some i. But if that's not true, it just means that the queue can grow one step above max before it is stopped from growing. I'll just remove the comment. --Jeremy _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
