diff options
| author | Sang-Heon Jeon <ekffu200098@gmail.com> | 2026-07-01 00:04:13 +0900 |
|---|---|---|
| committer | Mike Rapoport (Microsoft) <rppt@kernel.org> | 2026-07-01 09:22:41 +0300 |
| commit | 2db44dc8bdd3cfc933a3193724f4d38b03e35b80 (patch) | |
| tree | c50f741f91ec9691ceb936ced624a032e71fa9ad | |
| parent | c6af91e48594191f62ad3c1fcc8269b1eb539ef8 (diff) | |
| download | linux-2db44dc8bdd3cfc933a3193724f4d38b03e35b80.tar.gz linux-2db44dc8bdd3cfc933a3193724f4d38b03e35b80.zip | |
mm/hugetlb: remove unnecessary empty range check in hugetlb_bootmem_set_nodes()
hugetlb_bootmem_set_nodes() iterates the memory ranges with
for_each_mem_pfn_range() and calls node_set(nid, hugetlb_bootmem_nodes)
only when end_pfn > start_pfn. for_each_mem_pfn_range() never returns an
empty range, so start_pfn < end_pfn always.
Therefore the check is unnecessary, so remove it. start_pfn and end_pfn
are no longer used, so remove the local variables.
No functional change.
Signed-off-by: Sang-Heon Jeon <ekffu200098@gmail.com>
Reviewed-by: David Hildenbrand (Arm) <david@kernel.org>
Link: https://patch.msgid.link/20260630150413.1718632-9-ekffu200098@gmail.com
Signed-off-by: Mike Rapoport (Microsoft) <rppt@kernel.org>
| -rw-r--r-- | mm/hugetlb.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 571212b80835..4f7020962486 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -4444,15 +4444,12 @@ hugetlb_early_param("default_hugepagesz", default_hugepagesz_setup); void __init hugetlb_bootmem_set_nodes(void) { int i, nid; - unsigned long start_pfn, end_pfn; if (!nodes_empty(hugetlb_bootmem_nodes)) return; - for_each_mem_pfn_range(i, MAX_NUMNODES, &start_pfn, &end_pfn, &nid) { - if (end_pfn > start_pfn) - node_set(nid, hugetlb_bootmem_nodes); - } + for_each_mem_pfn_range(i, MAX_NUMNODES, NULL, NULL, &nid) + node_set(nid, hugetlb_bootmem_nodes); } void __init hugetlb_bootmem_alloc(void) |