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>

diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c
index 0db724aec5cb9a44d1efbbb1939b2d63198361dd..b22f6fbb5816e52ee862396cd5cf9b3178659f50 100644 (file)
--- a/mm/damon/paddr.c
+++ b/mm/damon/paddr.c
@@ -268,16 +268,15 @@ static inline unsigned long damon_pa_mark_accessed_or_deactivate(
                if (!folio)
                        continue;
 
-               if (damos_pa_filter_out(s, folio)) {
-                       folio_put(folio);
-                       continue;
-               }
+               if (damos_pa_filter_out(s, folio))
+                       goto put_folio;
 
                if (mark_accessed)
                        folio_mark_accessed(folio);
                else
                        folio_deactivate(folio);
                applied += folio_nr_pages(folio);
+put_folio:
                folio_put(folio);
        }
        return applied * PAGE_SIZE;