diff options
| author | Wei Yang <[email protected]> | 2024-06-19 01:06:09 +0000 |
|---|---|---|
| committer | Andrew Morton <[email protected]> | 2024-07-04 02:30:19 +0000 |
| commit | 861dd8b9e34fc3fc05762a952ad8dd701dc0f0f1 (patch) | |
| tree | ecbe4cb37b865314ed1bd54ad87d28ac4e336246 | |
| parent | mm/memory-failure: refactor log format in unpoison_memory (diff) | |
| download | kernel-861dd8b9e34fc3fc05762a952ad8dd701dc0f0f1.tar.gz kernel-861dd8b9e34fc3fc05762a952ad8dd701dc0f0f1.zip | |
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/[email protected]
Signed-off-by: Wei Yang <[email protected]>
Cc: Mike Rapoport (IBM) <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
| -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; |