> On 19 Mar 2017, at 14:40, Willy Tarreau <[email protected]> wrote: > > Hi, > > On Sat, Mar 18, 2017 at 01:12:09PM +0100, Willy Tarreau wrote: >> OK here's a temporary patch. It includes a revert of the previous one and >> adds a condition for the wake-up. At least it passes all my tests, including >> those involving synchronous connection reports. >> >> I'm not merging it yet as I'm wondering whether a reliable definitive >> solution should be done once for all (and backported) by addressing the >> root cause instead of constantly working around its consequences. > > And here come two patches as a replacement for this temporary one. They > are safer and have been done after throrough code review. I spotted a > small tens of dirty corner cases having accumulated over the years due > to the unclear meaning of the CO_FL_CONNECTED flag. They'll have to be > addressed, but the current patches protect against these corner cases. > They survived all tests involving delayed connections and checks with > and without all handshake combinations, with tcp (immediate and delayed > requests and responses) and http (immediate, delayed requests and responses > and pipelining). > > I'm resending the first one you already got Dmitry to make things easier > to follow for everyone. These three are to be applied on top of 1.7.3. I > still have a few other issues to deal with regarding 1.7 before doing a > new release (hopefully by the beginning of this week).
Thank a lot! I just incorporated the latest fixes to FreeBSD ports tree.
