diff options
| author | SJ Park <sj@kernel.org> | 2026-07-10 06:46:35 -0700 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2026-08-04 19:18:50 -0700 |
| commit | 3858025f48cf764c3c653b311a1fa1b2eb720a5c (patch) | |
| tree | 22237900eab2e140ca69606c18b100250a21a48a | |
| parent | cfe9e8c738d9acfa09973040e2dfa3fbd17c12be (diff) | |
| download | linux-3858025f48cf764c3c653b311a1fa1b2eb720a5c.tar.gz linux-3858025f48cf764c3c653b311a1fa1b2eb720a5c.zip | |
mm/damon/paddr: respect return_max_wsum
apply_probes() ops implementation in DAMON_PADDR is ignoring
return_max_wsum. Respect it.
Link: https://lore.kernel.org/20260710134651.18084-7-sj@kernel.org
Signed-off-by: SJ Park <sj@kernel.org>
Cc: David Hildenbrand <david@kernel.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Liam R. Howlett <liam@infradead.org>
Cc: Lorenzo Stoakes <ljs@kernel.org>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Mike Rapoport <rppt@kernel.org>
Cc: Suren Baghdasaryan <surenb@google.com>
Cc: Vlastimil Babka <vbabka@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
| -rw-r--r-- | mm/damon/paddr.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c index 3eba3564a31f..b85f88a7a38f 100644 --- a/mm/damon/paddr.c +++ b/mm/damon/paddr.c @@ -172,6 +172,7 @@ static unsigned int damon_pa_apply_probes(struct damon_ctx *ctx, struct damon_target *t; struct damon_region *r; struct damon_probe *p; + unsigned int max_wsum = 0; damon_for_each_target(t, ctx) { damon_for_each_region(r, t) { @@ -182,7 +183,6 @@ static unsigned int damon_pa_apply_probes(struct damon_ctx *ctx, if (set_samples) r->sampling_addr = damon_rand(ctx, r->ar.start, r->ar.end); - pa = damon_pa_phys_addr(r->sampling_addr, ctx->addr_unit); folio = damon_get_folio(PHYS_PFN(pa)); @@ -193,9 +193,12 @@ static unsigned int damon_pa_apply_probes(struct damon_ctx *ctx, } if (folio) folio_put(folio); + if (return_max_wsum) + max_wsum = max(damon_probe_hits_wsum(r, false, + ctx), max_wsum); } } - return 0; + return max_wsum; } /* |