mm/page_alloc: remove redundant pcp->free_count initialization in per_cpu_pages_init()

qiyeliu · akpm00 · commit 5922deb3ecc0 · 2025-09-13T16:55:02.000-07:00
In per_cpu_pages_init(), pcp->free_count is explicitly initialized to 0, but this is redundant because the entire struct is already zeroed by memset(pcp, 0, sizeof(*pcp)). Link: https://lkml.kernel.org/r/20250814071828.12036-1-ye.liu@linux.dev Signed-off-by: Ye Liu <liuye@kylinos.cn> Reviewed-by: Brendan Jackman <jackmanb@google.com> Acked-by: Johannes Weiner <hannes@cmpxchg.org> Reviewed-by: Zi Yan <ziy@nvidia.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
@@ -5946,7 +5946,6 @@ static void per_cpu_pages_init(struct per_cpu_pages *pcp, struct per_cpu_zonesta
 	pcp->high_min = BOOT_PAGESET_HIGH;
 	pcp->high_max = BOOT_PAGESET_HIGH;
 	pcp->batch = BOOT_PAGESET_BATCH;
-	pcp->free_count = 0;
 }
 
 static void __zone_set_pageset_high_and_batch(struct zone *zone, unsigned long high_min,

Original file line number	Diff line number	Diff line change
`@@ -5946,7 +5946,6 @@ static void per_cpu_pages_init(struct per_cpu_pages *pcp, struct per_cpu_zonesta`
`5946`	`5946`	`pcp->high_min = BOOT_PAGESET_HIGH;`
`5947`	`5947`	`pcp->high_max = BOOT_PAGESET_HIGH;`
`5948`	`5948`	`pcp->batch = BOOT_PAGESET_BATCH;`
`5949`		`- pcp->free_count = 0;`
`5950`	`5949`	`}`
`5951`	`5950`
`5952`	`5951`	`static void __zone_set_pageset_high_and_batch(struct zone *zone, unsigned long high_min,`