]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
selftests/mm: deduplicate test logging in test_mlock_lock()
authorMark Brown <broonie@kernel.org>
Thu, 15 May 2025 09:27:14 +0000 (11:27 +0200)
committerAndrew Morton <akpm@linux-foundation.org>
Sun, 1 Jun 2025 05:46:08 +0000 (22:46 -0700)
The mlock2-tests test_mlock_lock() test reports two test results with an
identical string, one reporitng if it successfully locked a block of
memory and another reporting if the lock is still present after doing an
unlock (following a similar pattern to other tests in the same program).
This confuses test automation since the test string is used to deduplicate
tests, change the post unlock test to report "Unlocked" instead like the
other tests to fix this.

Link: https://lkml.kernel.org/r/20250515-selftest-mm-mlock2-dup-v1-1-963d5d7d243a@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
Acked-by: Dev Jain <dev.jain@arm.com>
Cc: Shuah Khan <shuah@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
tools/testing/selftests/mm/mlock2-tests.c

index 7f0d50fa361dcf8726e46ef907a7b344e8cdc3bd..3e90ff37e336af2ed7f0127c6e1954ec74965873 100644 (file)
@@ -196,7 +196,7 @@ static void test_mlock_lock(void)
                ksft_exit_fail_msg("munlock(): %s\n", strerror(errno));
        }
 
-       ksft_test_result(!unlock_lock_check(map), "%s: Locked\n", __func__);
+       ksft_test_result(!unlock_lock_check(map), "%s: Unlocked\n", __func__);
        munmap(map, 2 * page_size);
 }