diff options
| author | Gavin Shan <[email protected]> | 2025-03-11 23:30:43 +0000 |
|---|---|---|
| committer | Andrew Morton <[email protected]> | 2025-03-18 05:07:01 +0000 |
| commit | 61659efdb35ce6c6ac7639342098f3c4548b794b (patch) | |
| tree | 7299fe37661b7ca9ace765360f6a15630bc38b58 /mm/sparse.c | |
| parent | mm/damon/sysfs-schemes: avoid Wformat-security warning on damon_sysfs_access_... (diff) | |
| download | kernel-61659efdb35ce6c6ac7639342098f3c4548b794b.tar.gz kernel-61659efdb35ce6c6ac7639342098f3c4548b794b.zip | |
drivers/base/memory: improve add_boot_memory_block()
Patch series "drivers/base/memory: Two cleanups", v3.
Two cleanups to drivers/base/memory.
This patch (of 2)L
It's unnecessary to count the present sections for the specified block
since the block will be added if any section in the block is present.
Besides, for_each_present_section_nr() can be reused as Andrew Morton
suggested.
Improve by using for_each_present_section_nr() and dropping the
unnecessary @section_count.
No functional changes intended.
Link: https://lkml.kernel.org/r/[email protected]
Link: https://lkml.kernel.org/r/[email protected]
Signed-off-by: Gavin Shan <[email protected]>
Acked-by: David Hildenbrand <[email protected]>
Acked-by: Oscar Salvador <[email protected]>
Cc: Danilo Krummrich <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Cc: "Rafael J. Wysocki" <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Diffstat (limited to 'mm/sparse.c')
| -rw-r--r-- | mm/sparse.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/mm/sparse.c b/mm/sparse.c index ee0234a77c7f..3c012cf83cc2 100644 --- a/mm/sparse.c +++ b/mm/sparse.c @@ -170,11 +170,6 @@ static void __section_mark_present(struct mem_section *ms, ms->section_mem_map |= SECTION_MARKED_PRESENT; } -#define for_each_present_section_nr(start, section_nr) \ - for (section_nr = next_present_section_nr(start-1); \ - section_nr != -1; \ - section_nr = next_present_section_nr(section_nr)) - static inline unsigned long first_present_section_nr(void) { return next_present_section_nr(-1); |