mm/memory_hotplug.c: use __pfn_to_section() instead of open-coding

qiwuchen · torvalds · commit 104049017b77 · 2020-04-07T10:43:41.000-07:00
Use __pfn_to_section() API instead of open-coding for better code readability. Signed-off-by: chenqiwu <chenqiwu@xiaomi.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Acked-by: David Hildenbrand <david@redhat.com> Link: http://lkml.kernel.org/r/1584345134-16671-1-git-send-email-qiwuchen55@gmail.com Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c
@@ -495,7 +495,7 @@ static void __remove_section(unsigned long pfn, unsigned long nr_pages,
 			     unsigned long map_offset,
 			     struct vmem_altmap *altmap)
 {
-	struct mem_section *ms = __nr_to_section(pfn_to_section_nr(pfn));
+	struct mem_section *ms = __pfn_to_section(pfn);
 
 	if (WARN_ON_ONCE(!valid_section(ms)))
 		return;

Original file line number	Diff line number	Diff line change
`@@ -495,7 +495,7 @@ static void __remove_section(unsigned long pfn, unsigned long nr_pages,`
`495`	`495`	`unsigned long map_offset,`
`496`	`496`	`struct vmem_altmap *altmap)`
`497`	`497`	`{`
`498`		`- struct mem_section *ms = __nr_to_section(pfn_to_section_nr(pfn));`
	`498`	`+ struct mem_section *ms = __pfn_to_section(pfn);`
`499`	`499`
`500`	`500`	`if (WARN_ON_ONCE(!valid_section(ms)))`
`501`	`501`	`return;`