On Mon, May 11, 2026 at 04:05:34PM +0200, David Hildenbrand (Arm) wrote: > We never free the ms->usage data for boot memory sections (see > section_deactivate()). And to identify whether ms->usage was allocated > from memblock, we simply identify it by looking at PG_reserved. > > Consequently, there is no need to mark ms->usage as MIX_SECTION_INFO. > Let's just stop doing that. > > Signed-off-by: David Hildenbrand (Arm) <[email protected]>
Reviewed-by: Mike Rapoport (Microsoft) <[email protected]> > --- > mm/bootmem_info.c | 12 +----------- > 1 file changed, 1 insertion(+), 11 deletions(-) > > diff --git a/mm/bootmem_info.c b/mm/bootmem_info.c > index cce1d560f094..0fa78db7fbc0 100644 > --- a/mm/bootmem_info.c > +++ b/mm/bootmem_info.c > @@ -38,10 +38,8 @@ void put_page_bootmem(struct page *page) > > static void __init register_page_bootmem_info_section(unsigned long > start_pfn) > { > - unsigned long mapsize, section_nr, i; > + unsigned long section_nr; > struct mem_section *ms; > - struct mem_section_usage *usage; > - struct page *page; > > start_pfn = SECTION_ALIGN_DOWN(start_pfn); > section_nr = pfn_to_section_nr(start_pfn); > @@ -50,14 +48,6 @@ static void __init > register_page_bootmem_info_section(unsigned long start_pfn) > if (!preinited_vmemmap_section(ms)) > register_page_bootmem_memmap(section_nr, pfn_to_page(start_pfn), > PAGES_PER_SECTION); > - > - usage = ms->usage; > - page = virt_to_page(usage); > - > - mapsize = PAGE_ALIGN(mem_section_usage_size()) >> PAGE_SHIFT; > - > - for (i = 0; i < mapsize; i++, page++) > - get_page_bootmem(section_nr, page, MIX_SECTION_INFO); > } > > void __init register_page_bootmem_info_node(struct pglist_data *pgdat) > > -- > 2.43.0 > -- Sincerely yours, Mike.
