Subject: Re: [PATCH 7/8] s390/mm: use free_reserved_page() in vmem_free_pages()

From: Heiko Carstens <[email-masked]>
Date: Mon, 11 May 2026 17:22:26 +0200

On Mon, May 11, 2026 at 04:24:16PM +0200, David Hildenbrand (Arm) wrote:
> 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-masked]>
> >> ---
> >>  arch/s390/mm/vmem.c | 3 +--
> >>  1 file changed, 1 insertion(+), 2 deletions(-)
...
> > 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-masked]

Heh, right, you can't expect from me that I would read more than one
patch of a series :)

Acked-by: Heiko Carstens <[email-masked]>