tree e468222b25749711c901d2e555cae9f2f9d63a82
parent d97856852fc6a8b4974d96c35bd740c9b9ebc349
author Willy Tarreau <w@1wt.eu> 1669037553 +0100
committer Christopher Faulet <cfaulet@haproxy.com> 1669373335 +0100

BUG/MINOR: server/idle: at least use atomic stores when updating max_used_conns

In 2.2, some idle conns usage metrics were added by commit cf612a045
("MINOR: servers: Add a counter for the number of currently used
connections."), which mentioned that the operation doesn't need to be
atomic since we're not seeking exact values. This is true but at least
we should use atomic stores to make sure not to cause invalid values
to appear on archs that wouldn't guarantee atomicity when writing an
int, such as writing two 16-bit words. This is pretty unlikely on our
targets but better keep the code safe against this.

This may be backported as far as 2.2.

(cherry picked from commit 9dc231a6b23fc7d5cf3c233b46e00b9e251325b4)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 51743eea8b1aca3c8ae7c9665976d21d348277ea)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 4df415bcbcc136ac744e5a52da11fab591f54c85)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>