mm/damon/tests/core-kunit: add test cases to damos_test_commit_filter()

damos_test_commit_filter() is covering only a single test case.  Extend it
to cover multiple combinations of inputs.

Link: https://lkml.kernel.org/r/20251111184415.141757-5-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Cc: Brendan Higgins <brendan.higgins@linux.dev>
Cc: David Gow <davidgow@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/mm/damon/tests/core-kunit.h b/mm/damon/tests/core-kunit.h
index 5052d8db96576971ffdd3175348a02d700c7bb4a..2770972b0a393bda00421d6f77e231915cec88c4 100644 (file)
--- a/mm/damon/tests/core-kunit.h
+++ b/mm/damon/tests/core-kunit.h
@@ -528,17 +528,58 @@ static void damos_test_commit_filter_for(struct kunit *test,
 
 static void damos_test_commit_filter(struct kunit *test)
 {
-       struct damos_filter src_filter = {
-               .type = DAMOS_FILTER_TYPE_ANON,
-               .matching = true,
-               .allow = true};
-       struct damos_filter dst_filter = {
+       struct damos_filter dst = {
                .type = DAMOS_FILTER_TYPE_ACTIVE,
                .matching = false,
                .allow = false,
        };
 
-       damos_test_commit_filter_for(test, &dst_filter, &src_filter);
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_ANON,
+                       .matching = true,
+                       .allow = true,
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_MEMCG,
+                       .matching = false,
+                       .allow = false,
+                       .memcg_id = 123,
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_YOUNG,
+                       .matching = true,
+                       .allow = true,
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_HUGEPAGE_SIZE,
+                       .matching = false,
+                       .allow = false,
+                       .sz_range = {.min = 234, .max = 345},
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_UNMAPPED,
+                       .matching = true,
+                       .allow = true,
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_ADDR,
+                       .matching = false,
+                       .allow = false,
+                       .addr_range = {.start = 456, .end = 567},
+                       });
+       damos_test_commit_filter_for(test, &dst,
+                       &(struct damos_filter){
+                       .type = DAMOS_FILTER_TYPE_TARGET,
+                       .matching = true,
+                       .allow = true,
+                       .target_idx = 6,
+                       });
 }
 
 static void damos_test_filter_out(struct kunit *test)