tree 6e0289ca9bf45fcdaf38cd5c4e54c6abd8e631e7
parent 67de95e2eb94b9f26620e6c3fc287687ee8e74d6
author Willy Tarreau <w@1wt.eu> 1614170772 +0100
committer Willy Tarreau <w@1wt.eu> 1615385476 +0100
encoding latin1

MEDIUM: streams: do not use the streams lock anymore

The lock was still used exclusively to deal with the concurrency between
the "show sess" release handler and a stream_new() or stream_free() on
another thread. All other accesses made by "show sess" are already done
under thread isolation. The release handler only requires to unlink its
node when stopping in the middle of a dump (error, timeout etc). Let's
just isolate the thread to deal with this case so that it's compatible
with the dump conditions, and remove all remaining locking on the streams.

This effectively kills the streams lock. The measured gain here is around
1.6% with 4 threads (374krps -> 380k).

(cherry picked from commit 49de68520e9dae3817885397373151aee4901c1b)
Signed-off-by: Willy Tarreau <w@1wt.eu>