]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: config: do not detect NUMA topology when cpu-map is configured
authorWilly Tarreau <w@1wt.eu>
Tue, 18 Jul 2023 15:45:21 +0000 (17:45 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 20 Jul 2023 09:01:09 +0000 (11:01 +0200)
commit67c99db0a7e4d6e9874cc76843778f790c9467dd
treed1c374c0a7e8555cff02124972530733bbdcbeae
parent71344176135b6e140ebc7c107ffe40ff9c3f79ba
BUG/MINOR: config: do not detect NUMA topology when cpu-map is configured

As documented, the NUMA auto-detection is not supposed to be used when
the CPU affinity was set either by taskset (already checked) or by a
cpu-map directive. However this check was missing, so that configs
having cpu-map entries would still first bind to a single node. In
practice it has no impact on correct configs since bindings will be
replaced. However for those where the cpu-map directive are not
exhaustive it will have the impact of binding those threads to one node,
which disagrees with the doc (and makes future evolutions significantly
more complicated).

This could be backported to 2.4 where numa-cpu-mapping was added, though
if nobody encountered this by then maybe we should only focus on recent
versions that are more NUMA-friendly (e.g. 2.8 only). This patch depends
on this previous commit that brings the function we rely on:

   MINOR: cpuset: add cpu_map_configured() to know if a cpu-map was found
src/cfgparse.c