github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-07-28 10:09:10 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

mm/damon/tests/vaddr-kunit: replace damon_add_region() with damon_set_regions()

Browse Source 
DAMON virtual address operation set (vaddr) unit tests is using
damon_add_region() for setup of DAMON monitoring target region boundaries
setup.  But, damon_set_regions() is designed for exactly the purpose.  All
other DAMON API callers use the function for the purpose.  Replace
damon_add_region() usage in the unit tests with damon_set_regions(), for
unifying the use case and reducing the maintenance cost.

Link: https://lore.kernel.org/20260522154026.80546-5-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Cc: Brendan Higgins <brendan.higgins@linux.dev>
Cc: Shuah Khan <shuah@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
This commit is contained in:
SeongJae Park 2026-05-22 08:40:15 -07:00 committed by Andrew Morton
parent cd036cc8c3
commit 9ace949ad8
1 changed files with 22 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
mm/damon/tests/vaddr-kunit.h
View File

@ -132,22 +132,35 @@ static void damon_do_test_apply_three_regions(struct kunit *test,
unsigned long *expected, int nr_expected)
{
struct damon_target *t;
struct damon_addr_range *ranges;
struct damon_region *r;
int i;
t = damon_new_target();
if (!t)
kunit_skip(test, "target alloc fail");
for (i = 0; i < nr_regions / 2; i++) {
r = damon_new_region(regions[i * 2], regions[i * 2 + 1]);
if (!r) {
damon_destroy_target(t, NULL);
kunit_skip(test, "region alloc fail");
}
damon_add_region(r, t);
}
damon_set_regions(t, three_regions, 3, DAMON_MIN_REGION_SZ);
ranges = kmalloc_array(nr_regions / 2, sizeof(*ranges), GFP_KERNEL);
if (!ranges) {
damon_destroy_target(t, NULL);
kunit_skip(test, "ranges alloc fail");
}
for (i = 0; i < nr_regions / 2; i++) {
ranges[i].start = regions[i * 2];
ranges[i].end = regions[i * 2 + 1];
}
if (damon_set_regions(t, ranges, nr_regions / 2,
DAMON_MIN_REGION_SZ)) {
kfree(ranges);
damon_destroy_target(t, NULL);
kunit_skip(test, "damon_set_regions() fail");
}
kfree(ranges);
if (damon_set_regions(t, three_regions, 3, DAMON_MIN_REGION_SZ)) {
damon_destroy_target(t, NULL);
kunit_skip(test, "second damon_set_regions() fail");
}
for (i = 0; i < nr_expected / 2; i++) {
r = __nth_region_of(t, i);