Re: [RFC 2/5] mm, page_alloc: calculate pageset high and batch once per zone

From: Oscar Salvador
Date: Thu Sep 10 2020 - 05:00:27 EST


On Mon, Sep 07, 2020 at 06:36:25PM +0200, Vlastimil Babka wrote:
> We currently call pageset_set_high_and_batch() for each possible cpu,
> which repeats the same calculations of high and batch values.
>
> Instead call it once per zone, and it applies the calculated values
> to all per-cpu pagesets of the zone.
>
> This also allows removing zone_pageset_init() and __zone_pcp_update() wrappers.
>
> No functional change.
>
> Signed-off-by: Vlastimil Babka <vbabka@xxxxxxx>

I like this, it simplifies the things.

Reviewed-by: Oscar Salvador <osalvador@xxxxxxx>

--
Oscar Salvador
SUSE L3