diff options
| author | Joshua Hahn <joshua.hahnjy@gmail.com> | 2025-01-13 13:01:56 -0800 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2025-01-25 20:22:41 -0800 |
| commit | f57f63b0f0fd7ed71987857d244f8490b2d6043c (patch) | |
| tree | 3a4c6367db801bb817093f083e773eaf1dca39c8 | |
| parent | bdbe1d7bc325dbe77335f39b265620f9accc0ae9 (diff) | |
| download | lwn-f57f63b0f0fd7ed71987857d244f8490b2d6043c.tar.gz lwn-f57f63b0f0fd7ed71987857d244f8490b2d6043c.zip | |
mm/damon/paddr: improve readability of damon_pa_stat
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].
[1] https://lore.kernel.org/all/20241219040327.61902-1-sj@kernel.org/
[2] https://lore.kernel.org/all/20241213215306.54778-1-sj@kernel.org/
Link: https://lkml.kernel.org/r/20250113210201.446051-1-joshua.hahnjy@gmail.com
Signed-off-by: Joshua Hahn <joshua.hahnjy@gmail.com>
Reviewed-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
| -rw-r--r-- | mm/damon/paddr.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c index bd8cfe10121b..0f9ae14f884d 100644 --- a/mm/damon/paddr.c +++ b/mm/damon/paddr.c @@ -513,11 +513,8 @@ static unsigned long damon_pa_stat(struct damon_region *r, struct damos *s, 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: addr += folio_size(folio); folio_put(folio); } |