Commit 9f86d624 authored by SeongJae Park's avatar SeongJae Park Committed by Linus Torvalds

mm/damon/vaddr-test: remove unnecessary variables

A couple of test functions in DAMON virtual address space monitoring
primitives implementation has unnecessary damon_ctx variables.  This
commit removes those.

Link: https://lkml.kernel.org/r/20211201150440.1088-7-sj@kernel.orgSigned-off-by: default avatarSeongJae Park <sj@kernel.org>
Cc: Brendan Higgins <brendanhiggins@google.com>
Cc: Shuah Khan <shuah@kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 044cd975
...@@ -135,7 +135,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test, ...@@ -135,7 +135,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test,
struct damon_addr_range *three_regions, struct damon_addr_range *three_regions,
unsigned long *expected, int nr_expected) unsigned long *expected, int nr_expected)
{ {
struct damon_ctx *ctx = damon_new_ctx();
struct damon_target *t; struct damon_target *t;
struct damon_region *r; struct damon_region *r;
int i; int i;
...@@ -145,7 +144,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test, ...@@ -145,7 +144,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test,
r = damon_new_region(regions[i * 2], regions[i * 2 + 1]); r = damon_new_region(regions[i * 2], regions[i * 2 + 1]);
damon_add_region(r, t); damon_add_region(r, t);
} }
damon_add_target(ctx, t);
damon_va_apply_three_regions(t, three_regions); damon_va_apply_three_regions(t, three_regions);
...@@ -154,8 +152,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test, ...@@ -154,8 +152,6 @@ static void damon_do_test_apply_three_regions(struct kunit *test,
KUNIT_EXPECT_EQ(test, r->ar.start, expected[i * 2]); KUNIT_EXPECT_EQ(test, r->ar.start, expected[i * 2]);
KUNIT_EXPECT_EQ(test, r->ar.end, expected[i * 2 + 1]); KUNIT_EXPECT_EQ(test, r->ar.end, expected[i * 2 + 1]);
} }
damon_destroy_ctx(ctx);
} }
/* /*
...@@ -298,8 +294,6 @@ static void damon_test_split_evenly_succ(struct kunit *test, ...@@ -298,8 +294,6 @@ static void damon_test_split_evenly_succ(struct kunit *test,
static void damon_test_split_evenly(struct kunit *test) static void damon_test_split_evenly(struct kunit *test)
{ {
struct damon_ctx *c = damon_new_ctx();
KUNIT_EXPECT_EQ(test, damon_va_evenly_split_region(NULL, NULL, 5), KUNIT_EXPECT_EQ(test, damon_va_evenly_split_region(NULL, NULL, 5),
-EINVAL); -EINVAL);
...@@ -307,8 +301,6 @@ static void damon_test_split_evenly(struct kunit *test) ...@@ -307,8 +301,6 @@ static void damon_test_split_evenly(struct kunit *test)
damon_test_split_evenly_succ(test, 0, 100, 10); damon_test_split_evenly_succ(test, 0, 100, 10);
damon_test_split_evenly_succ(test, 5, 59, 5); damon_test_split_evenly_succ(test, 5, 59, 5);
damon_test_split_evenly_fail(test, 5, 6, 2); damon_test_split_evenly_fail(test, 5, 6, 2);
damon_destroy_ctx(c);
} }
static struct kunit_case damon_test_cases[] = { static struct kunit_case damon_test_cases[] = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment