From: SeongJae Park Date: Mon, 10 Feb 2025 18:27:35 +0000 (-0800) Subject: mm/damon/core: do not call damos_walk_control->walk() if walk is completed X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=1d2368674549449bfa4bf8eb612d36c97fa9eddf;p=users%2Fjedix%2Flinux-maple.git mm/damon/core: do not call damos_walk_control->walk() if walk is completed damos_walk() invokes callback functions of schemes until all schemes finishes at least one round of walks. If there are multiple DAMOS schemes having different apply_interval, the callback functions for longer apply interval scheme will be called for more than a round of the walk. The behavior is different from the document (see damos_walk() kernel-doc comment), and not useful. Make the behavior be same to the documented one, by stopping invoking the callback if the walk for the given scheme is completed. Link: https://lkml.kernel.org/r/20250210182737.134994-3-sj@kernel.org Fixes: bf0eaba0ff9c ("mm/damon/core: implement damos_walk()") Signed-off-by: SeongJae Park Signed-off-by: Andrew Morton --- diff --git a/mm/damon/core.c b/mm/damon/core.c index 1d9025d14d83..4b865b2558d9 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -1453,6 +1453,9 @@ static void damos_walk_call_walk(struct damon_ctx *ctx, struct damon_target *t, { struct damos_walk_control *control; + if (s->walk_completed) + return; + mutex_lock(&ctx->walk_control_lock); control = ctx->walk_control; mutex_unlock(&ctx->walk_control_lock);