Gh0st
/
linux
mirror of https://github.com/torvalds/linux.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

mm/damon/tests/core-kunit: remove dynamic allocs on damos_test_commit_filter() 

Patch series "mm/damon/tests: add more tests for online parameters commit".

A DAMON feature called parameters "commit" allows DAMON API callers and
ABI users to update nearly every DAMON parameter while DAMON is running. 
This is being used for flexible DAMON use cases such as taking a snapshot
of the monitoring results with minimum overhead, or adjusting access-aware
system operations (DAMOS) for user-space driven auto-tuning or
investigations.

Compared to the usefulness of the feature and size of the implementation,
the test coverage is pretty small.  Only the filter commit part has a
single test case, namely damos_test_commit_filter().  Actually, we found
and fixed a few bugs of the feature in the past.  The single existing test
was also added to avoid reintroduction of a found bug.

Add more unit tests for the feature.

First four patches (1-4) refactor and extend the existing test for DAMOS
filter commit for multiple test cases.

Next three patches (5-7) add tests for DAMOS quota commit.

Next two patches (8 and 9) refactor damos_commit_dests() for ease of code
reading and test writing, and implement a new unit test of the function
that is being refactored in a test-friendly way.

Final two patches (10 and 11) further add new unit tests for
damos_commit() and damon_commit_target_regions().


This patch (of 11):

damos_test_commit_filter() is dynamically allocating test-purpose DAMOS
filters.  Allocation failure checks are making the code longer,
complicated, and difficult to extend for more test cases.  Refactor the
code to remove the dynamic allocation.

Link: https://lkml.kernel.org/r/20251111184415.141757-1-sj@kernel.org
Link: https://lkml.kernel.org/r/20251111184415.141757-2-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>
This commit is contained in:
SeongJae Park 2025-11-11 10:44:00 -08:00 committed by Andrew Morton
parent 7370f8e1b3
commit 37104286f9
1 changed files with 13 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
mm/damon/tests/core-kunit.h
View File

@ -499,23 +499,20 @@ static void damos_test_new_filter(struct kunit *test)
static void damos_test_commit_filter(struct kunit *test) static void damos_test_commit_filter(struct kunit *test)
{ {
struct damos_filter *src_filter, *dst_filter; struct damos_filter src_filter = {
.type = DAMOS_FILTER_TYPE_ANON,
.matching = true,
.allow = true};
struct damos_filter dst_filter = {
.type = DAMOS_FILTER_TYPE_ACTIVE,
.matching = false,
.allow = false,
};
src_filter = damos_new_filter(DAMOS_FILTER_TYPE_ANON, true, true); damos_commit_filter(&dst_filter, &src_filter);
if (!src_filter) KUNIT_EXPECT_EQ(test, dst_filter.type, src_filter.type);
kunit_skip(test, "src filter alloc fail"); KUNIT_EXPECT_EQ(test, dst_filter.matching, src_filter.matching);
dst_filter = damos_new_filter(DAMOS_FILTER_TYPE_ACTIVE, false, false); KUNIT_EXPECT_EQ(test, dst_filter.allow, src_filter.allow);
if (!dst_filter) {
damos_destroy_filter(src_filter);
kunit_skip(test, "dst filter alloc fail");
}
damos_commit_filter(dst_filter, src_filter);
KUNIT_EXPECT_EQ(test, dst_filter->type, src_filter->type);
KUNIT_EXPECT_EQ(test, dst_filter->matching, src_filter->matching);
KUNIT_EXPECT_EQ(test, dst_filter->allow, src_filter->allow);
damos_destroy_filter(src_filter);
damos_destroy_filter(dst_filter);
} }
static void damos_test_filter_out(struct kunit *test) static void damos_test_filter_out(struct kunit *test)