[Xcb] [PATCH 5/7] Add event queue splitting

Josh Triplett josh at joshtriplett.org
Fri Nov 8 07:45:34 PST 2013


On Wed, Nov 06, 2013 at 07:09:01PM -0800, Keith Packard wrote:
> Peter Harris <pharris at opentext.com> writes:
> 
> > Dude. It's not getting NAKed, it's getting "Woah, this is out of left
> > field, this is the first I've ever heard of it, can we please have a
> > couple of days to think about it before we ACK it?"ed.
> 
> Yeah, I've been running this code for months now and have mentioned it
> when presenting DRI3000, but didn't ever bring it directly to the XCB
> list because I implemented precisely what we'd already agreed to in
> principle -- the notion of a simple event filtering mechanism that split
> events off into separate queues
> 
> I didn't realize until recently that I *hadn't* even sent the patches to
> the list though; DRI3000 has so many modules involved. I have posted
> links to the work since last January, and to my knowledge, no-one has
> ever looked at any of the code in question, so XCB is at least not alone
> in that regard.
> 
> In any case, I'd like to implore all of you to consider a little
> deadline problem that I have today:
> 
>  * The Mesa freeze is Friday.
> 
>  * All of the DRI3/Present code has been reviewed and is ready to be
>    merged to master for that freeze.
> 
>  * However, it cannot be merged until a released version of XCB with
>    these APIs is available.
> 
>  * XCB is blocking the availability of a significant improvement to
>    Mesa.
> 
> If we miss that freeze, DRI3/Present and the opportunity for GL support
> in XCB will languish for at least another six months. And that would be
> a shame.

If you could send a v2 with the two naming changes proposed in this
thread, along with a rough handwave of how you think this could be
extended in the future to support more than one event type in one of
these side-queues without completely changing the API, that seems like
enough to go ahead and merge this.

- Josh Triplett


More information about the xorg-devel mailing list