diff options
| author | I Hsin Cheng <richard120310@gmail.com> | 2024-09-10 12:08:18 +0800 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2024-09-17 01:11:20 -0700 |
| commit | 5e06e08939df1cafef97a8e04f4b88c2806b538a (patch) | |
| tree | 10096ffb64f32444930c64c0e37912b3f44468b9 /lib/list-test.c | |
| parent | e620799c414a035dea1208bcb51c869744931dbb (diff) | |
| download | linux-5e06e08939df1cafef97a8e04f4b88c2806b538a.tar.gz linux-5e06e08939df1cafef97a8e04f4b88c2806b538a.tar.bz2 linux-5e06e08939df1cafef97a8e04f4b88c2806b538a.zip | |
list: test: increase coverage of list_test_list_replace*()
Increase the test coverage of list_test_list_replace*() by adding the
checks to compare the pointer of "a_new.next" and "a_new.prev" to make
sure a perfect circular doubly linked list is formed after the
replacement.
Link: https://lkml.kernel.org/r/20240910040818.65723-1-richard120310@gmail.com
Signed-off-by: I Hsin Cheng <richard120310@gmail.com>
Cc: David Gow <davidgow@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'lib/list-test.c')
| -rw-r--r-- | lib/list-test.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/list-test.c b/lib/list-test.c index 8d1d47a9fe9e..4f3dc75baec1 100644 --- a/lib/list-test.c +++ b/lib/list-test.c @@ -102,6 +102,8 @@ static void list_test_list_replace(struct kunit *test) /* now: [list] -> a_new -> b */ KUNIT_EXPECT_PTR_EQ(test, list.next, &a_new); KUNIT_EXPECT_PTR_EQ(test, b.prev, &a_new); + KUNIT_EXPECT_PTR_EQ(test, a_new.next, &b); + KUNIT_EXPECT_PTR_EQ(test, a_new.prev, &list); } static void list_test_list_replace_init(struct kunit *test) @@ -118,6 +120,8 @@ static void list_test_list_replace_init(struct kunit *test) /* now: [list] -> a_new -> b */ KUNIT_EXPECT_PTR_EQ(test, list.next, &a_new); KUNIT_EXPECT_PTR_EQ(test, b.prev, &a_new); + KUNIT_EXPECT_PTR_EQ(test, a_new.next, &b); + KUNIT_EXPECT_PTR_EQ(test, a_new.prev, &list); /* check a_old is empty (initialized) */ KUNIT_EXPECT_TRUE(test, list_empty_careful(&a_old)); |