From: Sang-Heon Jeon <ekffu200098@gmail.com>
To: rppt@kernel.org, Muchun Song <muchun.song@linux.dev>,
Oscar Salvador <osalvador@suse.de>,
Andrew Morton <akpm@linux-foundation.org>
Cc: linux-mm@kvack.org, Sang-Heon Jeon <ekffu200098@gmail.com>,
David Hildenbrand <david@kernel.org>,
linux-kernel@vger.kernel.org
Subject: [PATCH v3 8/8] mm/hugetlb: remove unnecessary empty range check in hugetlb_bootmem_set_nodes()
Date: Wed, 1 Jul 2026 00:04:13 +0900 [thread overview]
Message-ID: <20260630150413.1718632-9-ekffu200098@gmail.com> (raw)
In-Reply-To: <20260630150413.1718632-1-ekffu200098@gmail.com>
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>
---
mm/hugetlb.c | 7 ++-----
1 file 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)
--
2.43.0
prev parent reply other threads:[~2026-06-30 15:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-30 15:04 [PATCH v3 0/8] treewide: remove unnecessary invalid range checks in memblock iteration loops Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 1/8] arm64: mm: remove unreachable invalid range check in kasan_init_shadow() Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 2/8] LoongArch: remove unreachable invalid range check in kasan_init() Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 3/8] riscv: remove unreachable invalid range check in create_linear_mapping_page_table() Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 4/8] riscv: remove unreachable invalid range check in kasan_init() Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 5/8] ARM: " Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 6/8] powerpc64/kasan: Remove unreachable invalid range check in kasan_init_phys_region() Sang-Heon Jeon
2026-06-30 15:04 ` [PATCH v3 7/8] mm: remove unnecessary empty range check in early_calculate_totalpages() Sang-Heon Jeon
2026-06-30 15:04 ` Sang-Heon Jeon [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260630150413.1718632-9-ekffu200098@gmail.com \
--to=ekffu200098@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=david@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=muchun.song@linux.dev \
--cc=osalvador@suse.de \
--cc=rppt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox