diff options
| author | SJ Park <sj@kernel.org> | 2026-06-29 21:08:02 -0700 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2026-07-30 19:40:34 -0700 |
| commit | e33eb22d718b0bedb654fee9718e7c9f70a57fea (patch) | |
| tree | be0e33d59f96a09142191ab0a09be4b8fea50402 | |
| parent | 653902082cc80ee36300eb94c4499278f12b011c (diff) | |
mm/damon/core: remove damon_verify_reset_aggregated()
nr_accesses_bp is no longer being used in real use cases. Remove its
validation function.
Link: https://lore.kernel.org/20260630040812.149729-10-sj@kernel.org
Signed-off-by: SJ Park <sj@kernel.org>
Cc: Brendan Higgins <brendan.higgins@linux.dev>
Cc: "Masami Hiramatsu (Google)" <mhiramat@kernel.org>
Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: Shuah Khan <shuah@kernel.org>
Cc: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
| -rw-r--r-- | mm/damon/core.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/mm/damon/core.c b/mm/damon/core.c index 19606c959503..aa0b76ccac4c 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -2025,23 +2025,6 @@ int damos_walk(struct damon_ctx *ctx, struct damos_walk_control *control) return 0; } -#ifdef CONFIG_DAMON_DEBUG_SANITY -static void damon_verify_reset_aggregated(struct damon_region *r, - struct damon_ctx *c) -{ - WARN_ONCE(r->nr_accesses_bp != r->last_nr_accesses * 10000, - "nr_accesses_bp %u last_nr_accesses %u sis %lu %lu\n", - r->nr_accesses_bp, r->last_nr_accesses, - c->passed_sample_intervals, c->next_aggregation_sis); -} -#else -static void damon_verify_reset_aggregated(struct damon_region *r, - struct damon_ctx *c) -{ -} -#endif - - /* * Reset the aggregated monitoring results ('nr_accesses' of each region). */ @@ -2070,7 +2053,6 @@ static void kdamond_reset_aggregated(struct damon_ctx *c) r->nr_accesses = 0; for (i = 0; i < DAMON_MAX_PROBES; i++) r->probe_hits[i] = 0; - damon_verify_reset_aggregated(r, c); } ti++; } |