Skip to content

Commit dd41143

Browse files
Kefeng Wangakpm00
authored andcommitted
mm/damon/paddr: minor refactor of damon_pa_pageout()
Patch series "mm/damon/paddr: minor code improvement", v3. Unify folio_put() to make code more clear, and also fix minor issue in damon_pa_young(). This patch (of 3): Omit three lines by unified folio_put(), and make code more clear. Link: https://lkml.kernel.org/r/20230308083311.120951-1-wangkefeng.wang@huawei.com Link: https://lkml.kernel.org/r/20230308083311.120951-2-wangkefeng.wang@huawei.com Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com> Reviewed-by: SeongJae Park <sj@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
1 parent 22b8cc3 commit dd41143

1 file changed

Lines changed: 5 additions & 8 deletions

File tree

mm/damon/paddr.c

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -238,21 +238,18 @@ static unsigned long damon_pa_pageout(struct damon_region *r, struct damos *s)
238238
if (!folio)
239239
continue;
240240

241-
if (damos_pa_filter_out(s, folio)) {
242-
folio_put(folio);
243-
continue;
244-
}
241+
if (damos_pa_filter_out(s, folio))
242+
goto put_folio;
245243

246244
folio_clear_referenced(folio);
247245
folio_test_clear_young(folio);
248-
if (!folio_isolate_lru(folio)) {
249-
folio_put(folio);
250-
continue;
251-
}
246+
if (!folio_isolate_lru(folio))
247+
goto put_folio;
252248
if (folio_test_unevictable(folio))
253249
folio_putback_lru(folio);
254250
else
255251
list_add(&folio->lru, &folio_list);
252+
put_folio:
256253
folio_put(folio);
257254
}
258255
applied = reclaim_pages(&folio_list);

0 commit comments

Comments
 (0)