Hello Sailesh,
Thank you for this patch! >From next time, please send a new version of a patch as a new thread, not as a reply to the previous version. For the subject, please use mm/damon/tests/core-kunit: as the prefix. On Fri, 12 Jun 2026 11:53:37 +0530 Sailesh Nandanavanam <[email protected]> wrote: > Add a KUnit test to verify that damos_walk() rejects > new requests when walk_control_obsolete is set. > > Commit 33c3f6c2b48c ("mm/damon/core: fix damos_walk() vs > kdamond_fn() exit race") introduced walk_control_obsolete > to prevent a race condition where new requests could be > registered during kdamond shutdown and never handled. > > This test simulates the shutdown condition by setting > walk_control_obsolete and verifies that damos_walk() > returns -ECANCELED immediately. > > This validates the invariant introduced by the fix and > helps prevent regressions. > > Suggested-by: SeongJae Park <[email protected]> > Signed-off-by: Sailesh Nandanavanam <[email protected]> Other than the subject prefix, all looks good. Reviewed-by: SeongJae Park <[email protected]> I applied this patch to damon/next [1] tree after modifying the subject prefix as I suggested, assuming you don't mind. If you do mind that change, please let me know. We are now quite close to next merge window. We (mm community) want to focus on making mm.git more stabilized and therefore ready for the next merge window, rather than adding more changes that are not really urgent. I understand this patch is not really urgent. Hence, Andrew might not add this patch until next -rc1 release. In the case, I will request adding this to mm.git after next -rc1 release. So, no action from your side is needed for now. [1] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees Thanks, SJ [...]
