Re: [PATCH 11/21] mm: introduce freetype_t

[Dave Hansen]

On 9/24/25 07:59, Brendan Jackman wrote:
> @@ -2234,7 +2235,7 @@ static bool should_proactive_compact_node(pg_data_t *pgdat)
>  static enum compact_result __compact_finished(struct compact_control *cc)
>  {
>  	unsigned int order;
> -	const int migratetype = cc->migratetype;
> +	const freetype_t freetype = cc->freetype;

Just as I'm scanning this series at a high level, this patch looks too
big to me. There is too much mixing of mechanical changes like this
s/int/freetype_t/ and s/migratetype/freetype/ with new functionality.

I'd be looking for ways to split this up a lot more.