Skip to content

Commit 3572597

Browse files
committed
Merge tag 'fixes-2024-06-13' of git://git.kernel.org/pub/scm/linux/kernel/git/rppt/memblock
Pull memblock fixes from Mike Rapoport: "Fix validation of NUMA coverage. memblock_validate_numa_coverage() was checking for a unset node ID using NUMA_NO_NODE, but x86 used MAX_NUMNODES when no node ID was specified by buggy firmware. Update memblock to substitute MAX_NUMNODES with NUMA_NO_NODE in memblock_set_node() and use NUMA_NO_NODE in x86::numa_init()" * tag 'fixes-2024-06-13' of git://git.kernel.org/pub/scm/linux/kernel/git/rppt/memblock: x86/mm/numa: Use NUMA_NO_NODE when calling memblock_set_node() memblock: make memblock_set_node() also warn about use of MAX_NUMNODES
2 parents 2ccbdf4 + 3ac36aa commit 3572597

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

arch/x86/mm/numa.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -493,7 +493,7 @@ static void __init numa_clear_kernel_node_hotplug(void)
493493
for_each_reserved_mem_region(mb_region) {
494494
int nid = memblock_get_region_node(mb_region);
495495

496-
if (nid != MAX_NUMNODES)
496+
if (nid != NUMA_NO_NODE)
497497
node_set(nid, reserved_nodemask);
498498
}
499499

@@ -614,9 +614,9 @@ static int __init numa_init(int (*init_func)(void))
614614
nodes_clear(node_online_map);
615615
memset(&numa_meminfo, 0, sizeof(numa_meminfo));
616616
WARN_ON(memblock_set_node(0, ULLONG_MAX, &memblock.memory,
617-
MAX_NUMNODES));
617+
NUMA_NO_NODE));
618618
WARN_ON(memblock_set_node(0, ULLONG_MAX, &memblock.reserved,
619-
MAX_NUMNODES));
619+
NUMA_NO_NODE));
620620
/* In case that parsing SRAT failed. */
621621
WARN_ON(memblock_clear_hotplug(0, ULLONG_MAX));
622622
numa_reset_distance();

mm/memblock.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1339,6 +1339,10 @@ int __init_memblock memblock_set_node(phys_addr_t base, phys_addr_t size,
13391339
int start_rgn, end_rgn;
13401340
int i, ret;
13411341

1342+
if (WARN_ONCE(nid == MAX_NUMNODES,
1343+
"Usage of MAX_NUMNODES is deprecated. Use NUMA_NO_NODE instead\n"))
1344+
nid = NUMA_NO_NODE;
1345+
13421346
ret = memblock_isolate_range(type, base, size, &start_rgn, &end_rgn);
13431347
if (ret)
13441348
return ret;

0 commit comments

Comments
 (0)