Re: [PATCH 7/8] s390/mm: use free_reserved_page() in vmem_free_pages()
Heiko Carstens <[email protected]>
| Newsgroups | org.kernel.vger.sparclinux,org.kernel.vger.linux-kernel,org.kernel.vger.linux-s390,org.kvack.linux-mm,org.ozlabs.lists.linuxppc-dev |
|---|---|
| Message-ID | <[email protected]> |
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 protected]> > >> --- > >> 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 protected] Heh, right, you can't expect from me that I would read more than one patch of a series :) Acked-by: Heiko Carstens <[email protected]>