In damon_migrate_folio_list(), we're redeclaring folio inside the first while loop, but it just shadows the outer one. Since the second loop uses the outer folio anyway, the inner declaration is pointless. Remove it to consistently reuse the same variable throughout the function and improve readability. Signed-off-by: Enze Li --- mm/damon/ops-common.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/mm/damon/ops-common.c b/mm/damon/ops-common.c index 89c694f6e4fa..e59f5268d503 100644 --- a/mm/damon/ops-common.c +++ b/mm/damon/ops-common.c @@ -339,8 +339,6 @@ static unsigned int damon_migrate_folio_list(struct list_head *folio_list, LIST_HEAD(migrate_folios); while (!list_empty(folio_list)) { - struct folio *folio; - cond_resched(); folio = lru_to_folio(folio_list); base-commit: 5093dba1014c1d7f7e247fd118f0fa8f22136046 -- 2.43.0