]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: config: fix NUMA topology detection on FreeBSD
authorWilly Tarreau <w@1wt.eu>
Sat, 22 Apr 2023 17:26:07 +0000 (19:26 +0200)
committerWilly Tarreau <w@1wt.eu>
Sun, 23 Apr 2023 07:46:15 +0000 (09:46 +0200)
In 2.6-dev1, NUMA topology detection was enabled on FreeBSD with commit
f5d48f8b3 ("MEDIUM: cfgparse: numa detect topology on FreeBSD."). But
it suffers from a minor bug which is that it forgets to check for the
number of domains and always emits a confusing warning indicating that
multiple sockets were found while it's not the case.

This can be backported to 2.6.

src/cfgparse.c

index a8b6bb86930c43e09b742bd2cfcc3880e1d04287..9855ee1cc511df9a78e46fb124294fde710893ec 100644 (file)
@@ -2701,6 +2701,9 @@ static int numa_detect_topology()
        BUG_ON(ndomains > MAXMEMDOM);
        ha_cpuset_zero(&node_cpu_set);
 
+       if (ndomains < 2)
+               goto leave;
+
        /*
         * We retrieve the first active valid CPU domain
         * with active cpu and binding it, we returns
@@ -2726,7 +2729,7 @@ static int numa_detect_topology()
                }
                break;
        }
-
+ leave:
        return ha_cpuset_count(&node_cpu_set);
 }