Skip to content

Commit b6993be

Browse files
Kefeng Wangakpm00
authored andcommitted
mm/damon/paddr: minor refactor of damon_pa_mark_accessed_or_deactivate()
Omit one line by unified folio_put(), and make code more clear. Link: https://lkml.kernel.org/r/20230308083311.120951-3-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 dd41143 commit b6993be

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

mm/damon/paddr.c

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -268,16 +268,15 @@ static inline unsigned long damon_pa_mark_accessed_or_deactivate(
268268
if (!folio)
269269
continue;
270270

271-
if (damos_pa_filter_out(s, folio)) {
272-
folio_put(folio);
273-
continue;
274-
}
271+
if (damos_pa_filter_out(s, folio))
272+
goto put_folio;
275273

276274
if (mark_accessed)
277275
folio_mark_accessed(folio);
278276
else
279277
folio_deactivate(folio);
280278
applied += folio_nr_pages(folio);
279+
put_folio:
281280
folio_put(folio);
282281
}
283282
return applied * PAGE_SIZE;

0 commit comments

Comments
 (0)