[PATCH] Cannot wait for user lock in sync_while_locked

Jamey Sharp jamey at minilop.net
Sat Mar 10 21:05:21 PST 2012


On Sat, Mar 10, 2012 at 4:04 PM, Keith Packard <keithp at keithp.com> wrote:
> On Sat, 10 Mar 2012 14:31:52 -0800, Jamey Sharp <jamey at minilop.net> wrote:
>> Although, in "Restore Xlib semantics", it would be nice if you'd fix
>> up the "It appears that classic Xlib respected user locks" comment. I
>> think it suffices to delete the final sentence, "So we'll choose to
>> let the thread that got in first consume events, despite the later
>> thread's user locks."
>
> something like:
>
> -                        * deadlock. So we'll choose to let the thread
> -                        * that got in first consume events, despite the
> -                        * later thread's user locks. */
> +                        * deadlock. So we let the thread got in first read
> +                        * events, then signal other waiting threads and go
> +                        * back to check for a user lock */

Looks great; just insert "let the thread [that] got in first" and push. :-)

>> Thanks for the careful review and detailed summary!
>
> Xlib locking code is hard. Let's go rocketing!

OMG rocket science is so much easier than Xlib surgery.

Jamey


More information about the xorg-devel mailing list