Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 28 Mar 2021 01:43:51 GMT
From:      Brandon Bergren <bdragon@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: bd94c8ab29c3 - main - [PowerPC] Fix NUMA checking for powernv
Message-ID:  <202103280143.12S1hptT005562@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by bdragon:

URL: https://cgit.FreeBSD.org/src/commit/?id=bd94c8ab29c3162bbb43973ee77ce245fe157fef

commit bd94c8ab29c3162bbb43973ee77ce245fe157fef
Author:     Brandon Bergren <bdragon@FreeBSD.org>
AuthorDate: 2021-03-28 01:41:45 +0000
Commit:     Brandon Bergren <bdragon@FreeBSD.org>
CommitDate: 2021-03-28 01:42:49 +0000

    [PowerPC] Fix NUMA checking for powernv
    
    At this point in startup, vm_ndomains has not been initialized. Switch
    to checking kenv instead.
    
    Fixes incorrect NUMA information being set on multi-domain systems like
    Talos II.
    
    Submitted by:   jhibbits
    MFC after:      2 weeks
---
 sys/powerpc/powernv/platform_powernv.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/sys/powerpc/powernv/platform_powernv.c b/sys/powerpc/powernv/platform_powernv.c
index 434b642a66a8..d7acc544c2ed 100644
--- a/sys/powerpc/powernv/platform_powernv.c
+++ b/sys/powerpc/powernv/platform_powernv.c
@@ -532,7 +532,9 @@ powernv_node_numa_domain(platform_t platform, phandle_t node)
 #ifndef NUMA
 	return (0);
 #endif
-	if (vm_ndomains == 1)
+	i = 0;
+	TUNABLE_INT_FETCH("vm.numa.disabled", &i);
+	if (i)
 		return (0);
 
 	res = OF_getencprop(node, "ibm,associativity",



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202103280143.12S1hptT005562>