[PATCH] Cannot wait for user lock in sync_while_locked

Keith Packard keithp at keithp.com
Sat Mar 10 16:04:33 PST 2012


<#part sign=pgpmime>
On Sat, 10 Mar 2012 14:31:52 -0800, Jamey Sharp <jamey at minilop.net> wrote:

> Reviewed-by: Jamey Sharp <jamey at minilop.net>

Thanks!

> 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 */

> Thanks for the careful review and detailed summary!

Xlib locking code is hard. Let's go rocketing!

-- 
keith.packard at intel.com


More information about the xorg-devel mailing list