tree 4788d3f635b02d2a51c0ff6c2b1ff9b09329f208
parent 98c8910be748fa3a3f3ff6bff77b89177f5fd350
author Willy Tarreau <w@1wt.eu> 1624514664 +0200
committer Willy Tarreau <w@1wt.eu> 1624524751 +0200
encoding latin1

MEDIUM: queue: take the proxy lock only during the px queue accesses

There's no point keeping the proxy lock held for a long time, it's
only needed when checking the proxy's queue, and keeping it prevents
multiple servers from dequeuing in parallel. Let's move it into
pendconn_process_next_strm() and release it ASAP. The pendconn
remains under the server queue lock's protection, guaranteeing that
no stream will release it while it's being touched.

For roundrobin, the performance increases by 76% (327k to 575k) on
16 threads. Even with a single server and maxconn=100, the performance
increases from 398 to 496 kreq/s. For leastconn, almost no change is
visible (less than one percent) but this is expected since most of the
time there is spent in fwlc_reposition() and fwlc_get_next_server().