]> git.ipfire.org Git - thirdparty/haproxy.git/commit
OPTIM: pool: split the read_mostly from read_write parts in pool_head
authorWilly Tarreau <w@1wt.eu>
Tue, 20 Dec 2022 10:54:35 +0000 (11:54 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 20 Dec 2022 13:51:12 +0000 (14:51 +0100)
commit4dd33d9c322d3be167c3d672aebf6108f4f7889b
tree4b2392f68620824c6591afc1e125ff6e6b6f6846
parenta8b7684319a48110afb61d50f004566886a6baf1
OPTIM: pool: split the read_mostly from read_write parts in pool_head

Performance profiling on a 48-thread machine showed a lot of time spent
in pool_free(), precisely at the point where pool->limit was retrieved.
And the reason is simple. Some parts of the pool_head are heavily updated
only when facing a cache miss ("allocated", "used", "needed_avg"), while
others are always accessed (limit, flags, size). The fact that both
entries were stored into the same cache line makes it very difficult for
each thread to access these precious info even when working with its own
cache.

By just splitting the fields apart, a test on QUIC (which stresses pools
a lot) more than doubled performance from 42 Gbps to 96 Gbps!

Given that the patch only reorders fields and addresses such a significant
contention, it should be backported to 2.7 and 2.6.
include/haproxy/pool-t.h