Re: [PATCH v4 02/21] mm/memory_hotplug: Move {get,put}_page_bootmem() to bootmem_info.c

From: Oscar Salvador
Date: Mon Nov 16 2020 - 08:52:50 EST


On Fri, Nov 13, 2020 at 06:59:33PM +0800, Muchun Song wrote:
> In the later patch, we will use {get,put}_page_bootmem() to initialize
> the page for vmemmap or free vmemmap page to buddy. So move them out of
> CONFIG_MEMORY_HOTPLUG_SPARSE. This is just code movement without any
> functional change.
>
> Signed-off-by: Muchun Song <songmuchun@xxxxxxxxxxxxx>
> Acked-by: Mike Kravetz <mike.kravetz@xxxxxxxxxx>

The change itself makes sense.
I tried to check the possibles scenarios with different CONFIG_ options
toggled, but I will trust you that you ran the checks :-)

Reviewed-by: Oscar Salvador <osalvador@xxxxxxx>


--
Oscar Salvador
SUSE L3