please test: Xlib/XCB fixes

Jamey Sharp jamey at minilop.net
Mon May 10 17:00:09 PDT 2010


I guess I'm not going to get much review or testing of these patches
while they're on a branch, so I've just pushed them to libX11 master.
I've been running libX11 patched this way for about three weeks with
no apparent problems, and the patches make multi-threaded ico
completely work as far as I can tell, so I think they're a win.

Testing and review would still be welcome though. :-)

Jamey

On Sun, Apr 18, 2010 at 11:33 PM, Jamey Sharp <jamey at minilop.net> wrote:
> Josh and I sorted out a bunch of Xlib bugs this week, especially in
> support for multi-threaded applications. I'm embarrassed that I hadn't
> realized the thread support was so bad--but there are so few threaded
> Xlib apps, and even fewer that are open source.
>
> One important fix is in libxcb master. Several more fixes are in libX11
> master, but my invasive rework of xcb_io.c is in an
> xlib-xcb-thread-fixes branch pending testing and/or review.
>
> http://cgit.freedesktop.org/xorg/lib/libX11/log/?h=xlib-xcb-thread-fixes
>
> These changes need testing to make sure I didn't introduce regressions
> for the single-threaded applications that were already working. And of
> course, if anyone has multi-threaded apps that were failing, please
> retest them and let me know if these patches helped.
>
> Also, please review these patches. The final patch of the libX11 series
> adds lots of comments--including things I had to explain to Josh while
> he was reviewing--and the new invariants it maintains are relatively
> straightforward, so I hope others can make sense of it too.
>
> While most of the bugs we fixed affected only clients using the
> connection from multiple threads, some additional fixes affect
> single-threaded applications as well. I believe at least these bugs have
> been solved by these changes:
>
> - https://bugs.freedesktop.org/show_bug.cgi?id=26545
> - https://bugs.freedesktop.org/show_bug.cgi?id=27595
> - http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=535534
>
> If you can take some time to go through your favorite bug tracking
> system looking for other bugs that sound relevant, please do. Since I
> have all this stuff swapped in now, this would be a good time to bring
> more Xlib/XCB-related bug reports to my attention... :-)
>
> Hoping to be Done With Xlib any day now,
> Jamey
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
>
> iEYEARECAAYFAkvL+R4ACgkQp1aplQ4I9mXHsACfYePmzD1F226+WMxao5OsygZu
> /agAniScnI4oUYDbvS5MN/A5Mc79cg6z
> =hg6H
> -----END PGP SIGNATURE-----
>
>


More information about the xorg-devel mailing list