diff options
| author | Wei Yang <richard.weiyang@gmail.com> | 2024-06-19 01:06:09 +0000 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2024-07-03 19:30:19 -0700 |
| commit | 861dd8b9e34fc3fc05762a952ad8dd701dc0f0f1 (patch) | |
| tree | ecbe4cb37b865314ed1bd54ad87d28ac4e336246 /mm/sparse.c | |
| parent | 5cea5666e4b556f4daa414a7379790ce8d225a48 (diff) | |
mm/sparse: nr_pages won't be 0
Function subsection_map_init() is only used in free_area_init() in the
loop of for_each_mem_pfn_range(). And we are sure in each iteration of
for_each_mem_pfn_range(), start_pfn < end_pfn.
So nr_pages is not possible to be 0 and we can remove the check.
Link: https://lkml.kernel.org/r/20240619010612.20740-1-richard.weiyang@gmail.com
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Cc: Mike Rapoport (IBM) <rppt@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'mm/sparse.c')
| -rw-r--r-- | mm/sparse.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/mm/sparse.c b/mm/sparse.c index 8724a06d8128..e4b830091d13 100644 --- a/mm/sparse.c +++ b/mm/sparse.c @@ -195,9 +195,6 @@ void __init subsection_map_init(unsigned long pfn, unsigned long nr_pages) int end_sec_nr = pfn_to_section_nr(pfn + nr_pages - 1); unsigned long nr, start_sec_nr = pfn_to_section_nr(pfn); - if (!nr_pages) - return; - for (nr = start_sec_nr; nr <= end_sec_nr; nr++) { struct mem_section *ms; unsigned long pfns; |