Re: [PATCH 08/10] mm: page_alloc: set migratetype inside move_freepages()

From: Vlastimil Babka
Date: Tue Mar 26 2024 - 11:43:45 EST


On 3/20/24 7:02 PM, Johannes Weiner wrote:
> From: Zi Yan <ziy@xxxxxxxxxx>
>
> This avoids changing migratetype after move_freepages() or
> move_freepages_block(), which is error prone. It also prepares for
> upcoming changes to fix move_freepages() not moving free pages
> partially in the range.
>
> Signed-off-by: Zi Yan <ziy@xxxxxxxxxx>
> Signed-off-by: Johannes Weiner <hannes@xxxxxxxxxxx>

Reviewed-by: Vlastimil Babka <vbabka@xxxxxxx>

BTW noticed in -mm this has R-b: Zi Yan which is odd as he's the author. In
the subthread for patch 9/10 Zi posted a R-b for patch 9/10 and its fixup,
not this one :)