On 5/11/26 16:21, Heiko Carstens wrote: > On Mon, May 11, 2026 at 04:05:35PM +0200, David Hildenbrand (Arm) wrote: >> We never select CONFIG_HAVE_BOOTMEM_INFO_NODE on s390. Therefore, >> free_bootmem_page() nowadays always translates to free_reserved_page(). >> >> Let's use free_reserved_page() to replace the free_bootmem_page() loop. >> We can stop including bootmem_info.h. >> >> Likely, vmemmap freeing code could be factored out into the core in the >> future. >> >> Signed-off-by: David Hildenbrand (Arm) <[email protected]> >> --- >> arch/s390/mm/vmem.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/arch/s390/mm/vmem.c b/arch/s390/mm/vmem.c >> index eeadff45e0e1..d8b2a60e0c33 100644 >> --- a/arch/s390/mm/vmem.c >> +++ b/arch/s390/mm/vmem.c >> @@ -4,7 +4,6 @@ >> */ >> >> #include <linux/memory_hotplug.h> >> -#include <linux/bootmem_info.h> >> #include <linux/cpufeature.h> >> #include <linux/memblock.h> >> #include <linux/pfn.h> >> @@ -51,7 +50,7 @@ static void vmem_free_pages(unsigned long addr, int order, >> struct vmem_altmap *a >> if (PageReserved(page)) { >> /* allocated from memblock */ >> while (nr_pages--) >> - free_bootmem_page(page++); >> + free_reserved_page(page++); > > What about the implicit call of kmemleak_free_part_phys() which gets > removed with this?
Thanks for taking a look! See patch #4: https://lore.kernel.org/r/[email protected] -- Cheers, David
