[PATCH 1/2 v5] mieq: Provide better adaptability and diagnostics during mieq overflow
Jeremy Huddleston
jeremyhu at apple.com
Sun Oct 16 22:35:37 PDT 2011
On Oct 16, 2011, at 10:30 PM, Keith Packard wrote:
> On Sun, 16 Oct 2011 22:14:31 -0700, Jeremy Huddleston <jeremyhu at apple.com> wrote:
>
>> + if ((miEventQueue.tail - miEventQueue.head) % miEventQueue.nevents >= (miEventQueue.nevents >> 1) &&
>
> I missed this use of '%'. And, this seems to be growing the queue
> whenever it's more than half full? Is that too pessimistic?
Re: %
Yeah, I missed that too when I was fixing the other one. Will address. I'm tired of spinning new versions of this patch and will wait for additional feedback before sending the final one.
Re: pessimism
I dunno. They're your cats. You tell me. As much as I bang on the keyboard, I can't get the queue to resize larger than 32 with this scheme (starting at 2).
Maybe just resizing if all we have left is 2 * reserved space would be a good idea.
--Jeremy
More information about the xorg-devel
mailing list