Re: [PATCH 2/9] mm, page_alloc: remove boot pageset initialization from memory hotplug

From: Mel Gorman
Date: Fri Jul 14 2017 - 05:39:50 EST


On Fri, Jul 14, 2017 at 09:59:59AM +0200, Michal Hocko wrote:
> From: Michal Hocko <mhocko@xxxxxxxx>
>
> boot_pageset is a boot time hack which gets superseded by normal
> pagesets later in the boot process. It makes zero sense to reinitialize
> it again and again during memory hotplug.
>
> Signed-off-by: Michal Hocko <mhocko@xxxxxxxx>

You could also go slightly further and remove the batch parameter to
setupo_pageset because it's always 0. Otherwise

Acked-by: Mel Gorman <mgorman@xxxxxxx>

--
Mel Gorman
SUSE Labs