Re: [PATCH 6/8] mm/bootmem_info: stop marking mem_section_usage as MIX_SECTION_INFO

[Mike Rapoport]

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) <david@xxxxxxxxxx>

Reviewed-by: Mike Rapoport (Microsoft) <rppt@xxxxxxxxxx>

> ---
>  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.