]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
selftests/mm: guard-regions: Use SKIP() instead of ksft_exit_skip()
authorwang lian <lianux.mm@gmail.com>
Thu, 17 Jul 2025 13:18:57 +0000 (21:18 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 25 Jul 2025 02:12:42 +0000 (19:12 -0700)
To ensure only the current test is skipped on permission failure, instead
of terminating the entire test binary.

Link: https://lkml.kernel.org/r/20250717131857.59909-3-lianux.mm@gmail.com
Signed-off-by: wang lian <lianux.mm@gmail.com>
Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Acked-by: David Hildenbrand <david@redhat.com>
Reviewed-by: Mark Brown <broonie@kernel.org>
Reviewed-by: Zi Yan <ziy@nvidia.com>
Reviewed-by: Wei Yang <richard.weiyang@gmail.com>
Cc: Christian Brauner <brauner@kernel.org>
Cc: Jann Horn <jannh@google.com>
Cc: Kairui Song <ryncsn@gmail.com>
Cc: Liam Howlett <liam.howlett@oracle.com>
Cc: SeongJae Park <sj@kernel.org>
Cc: Shuah Khan <shuah@kernel.org>
Cc: Vlastimil Babka <vbabka@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
tools/testing/selftests/mm/guard-regions.c

index 4b76e72e70531c94fdbacc64d5195678839de66e..b0d42eb04e3ae86d298492e23f5c85e24f387537 100644 (file)
@@ -575,7 +575,7 @@ TEST_F(guard_regions, process_madvise)
 
        /* OK we don't have permission to do this, skip. */
        if (count == -1 && errno == EPERM)
-               ksft_exit_skip("No process_madvise() permissions, try running as root.\n");
+               SKIP(return, "No process_madvise() permissions, try running as root.\n");
 
        /* Returns the number of bytes advised. */
        ASSERT_EQ(count, 6 * page_size);