]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: limits: compute_ideal_maxconn: don't cap remain if fd_hard_limit=0
authorValentine Krasnobaeva <vkrasnobaeva@haproxy.com>
Tue, 18 Mar 2025 15:33:54 +0000 (16:33 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 18 Mar 2025 16:37:33 +0000 (17:37 +0100)
commit060f441199aa97d9735dd553bafd231ca615f723
tree92135ef0d33d79a43c51bdcd97a41bc65b9b9683
parent6336b636f7ba108880b82093ee54de41ca561165
BUG/MINOR: limits: compute_ideal_maxconn: don't cap remain if fd_hard_limit=0

'global.fd_hard_limit' stays uninitialized, if haproxy is started with -m
(global.rlimit_memmax). 'remain' is the MAX between soft and hard process fd
limits. It will be always bigger than 'global.fd_hard_limit' (0) in this case.

So, if we reassign 'remain' to the 'global.fd_hard_limit' unconditionally,
calculated then 'maxconn' will be even negative and the DEFAULT_MAXCONN (100)
will be set as the 'ideal_maxconn'.

During the 'global.maxconn' calculations in set_global_maxconn(), if the
provided 'global.rlimit_memmax' is quite big, system will refuse to calculate
based on its 'global.maxconn' and we will do a fallback to the 'ideal_maxconn',
which is 100.

Same problem for the configs with SSL frontends and backends.

This fixes the issue #2899.

This should be backported to v3.1.0.
src/limits.c