damon_pa_stat contains an unnecessary goto statement, and the if/else can be re-written to be more readable. This patch is written on top of SJ's patch series [1], which in turn is written on top of another one of his series [2]. Signed-off-by: Joshua Hahn <joshua.hahnjy@xxxxxxxxx> [1] https://lore.kernel.org/all/20241219040327.61902-1-sj@xxxxxxxxxx/ [2] https://lore.kernel.org/all/20241213215306.54778-1-sj@xxxxxxxxxx/ --- mm/damon/paddr.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c index b0c283808ba6..8cdf8320f9a5 100644 --- a/mm/damon/paddr.c +++ b/mm/damon/paddr.c @@ -509,11 +509,8 @@ static unsigned long damon_pa_stat(struct damon_region *r, struct damos *s, if (!folio) continue; - if (damos_pa_filter_out(s, folio)) - goto put_folio; - else + if (!damos_pa_filter_out(s, folio)) *sz_filter_passed += folio_size(folio); -put_folio: folio_put(folio); } return 0; base-commit: 2c0573ae5b4c9c0f9d81fa0138ed0014e1baf41b -- 2.43.5
