From: Kefeng Wang <wangkefeng.wang@huawei.com>
Date: Wed, 8 Mar 2023 08:33:10 +0000 (+0800)
Subject: mm/damon/paddr: minor refactor of damon_pa_mark_accessed_or_deactivate()
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=b6993be23601c8bc992dc9743fbf78c1ff5d6b6a;p=linux.git

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 0db724aec5cb9..b22f6fbb5816e 100644
--- 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;