Re: [RFC PATCH 10/14] mm/damon/vaddr: put pid in cleanup_target()

[SeongJae Park <sj@kernel.org>]

On Sun,  6 Jul 2025 13:00:14 -0700 SeongJae Park <sj@kernel.org> wrote:

> Implement cleanup_target() callback for [f]vaddr, which calls put_pid()
> for each target that will be destroyed.  Also remove redundant put_pid()
> calls in core and sysfs, which was required to be done redundantly due
> to the lack of such self cleanup in vaddr.
> 
> Signed-off-by: SeongJae Park <sj@kernel.org>
> ---
>  mm/damon/core.c  |  2 --
>  mm/damon/sysfs.c | 10 ++--------
>  mm/damon/vaddr.c |  6 ++++++
>  3 files changed, 8 insertions(+), 10 deletions(-)

This patch forgot removing put_pid() from DAMON sample modules.  I will add
below fixup on the next version.


Thanks,
SJ

[...]
---------------------- >8 --------------------------
diff --git a/samples/damon/prcl.c b/samples/damon/prcl.c
index d3215699a5f4..60a84fffed57 100644
--- a/samples/damon/prcl.c
+++ b/samples/damon/prcl.c
@@ -120,8 +120,6 @@ static void damon_sample_prcl_stop(void)
                damon_stop(&ctx, 1);
                damon_destroy_ctx(ctx);
        }
-       if (target_pidp)
-               put_pid(target_pidp);
 }

 static bool init_called;
diff --git a/samples/damon/wsse.c b/samples/damon/wsse.c
index fb24190d2fc0..82a014c53cec 100644
--- a/samples/damon/wsse.c
+++ b/samples/damon/wsse.c
@@ -100,8 +100,6 @@ static void damon_sample_wsse_stop(void)
                damon_stop(&ctx, 1);
                damon_destroy_ctx(ctx);
        }
-       if (target_pidp)
-               put_pid(target_pidp);
 }

 static bool init_called;