On Sat, 2017-03-25 at 21:17 +0100, Florian Weimer wrote: > * Torvald Riegel: > > > + bool registered_while_in_write_phase = false; > > if (__glibc_likely ((r & PTHREAD_RWLOCK_WRPHASE) == 0)) > > return 0; > > + else > > + registered_while_in_write_phase = true; > > Sorry, this doesn't look quite right. Isn't > registered_while_in_write_phase always true? Attached is a v2 patch. It's the same logic, but bigger. Most of this increase is due to reformatting, but I also adapted some of the comments. I get two failures, but I guess these are either due to the bad internet connectivity I currently have, or something at the resolver. FAIL: resolv/mtrace-tst-leaks FAIL: resolv/tst-leaks