selftests/bpf: Clean up dynptr prog_tests
authorJoanne Koong <joannelkoong@gmail.com>
Tue, 14 Feb 2023 05:13:32 +0000 (21:13 -0800)
committerAlexei Starovoitov <ast@kernel.org>
Tue, 14 Feb 2023 05:42:29 +0000 (21:42 -0800)
Clean up prog_tests/dynptr.c by removing the unneeded "expected_err_msg"
in the dynptr_tests struct, which is a remnant from converting the fail
tests cases to use the generic verification tester.

Signed-off-by: Joanne Koong <joannelkoong@gmail.com>
Link: https://lore.kernel.org/r/20230214051332.4007131-2-joannelkoong@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/prog_tests/dynptr.c

index 7faaf6d9e0d475e25235a66a5627d57f62cf1121..b99264ec0d9ca3d1578d7e22f1b53d7df6dedc96 100644 (file)
@@ -5,14 +5,10 @@
 #include "dynptr_fail.skel.h"
 #include "dynptr_success.skel.h"
 
-static struct {
-       const char *prog_name;
-       const char *expected_err_msg;
-} dynptr_tests[] = {
-       /* success cases */
-       {"test_read_write", NULL},
-       {"test_data_slice", NULL},
-       {"test_ringbuf", NULL},
+static const char * const success_tests[] = {
+       "test_read_write",
+       "test_data_slice",
+       "test_ringbuf",
 };
 
 static void verify_success(const char *prog_name)
@@ -53,11 +49,11 @@ void test_dynptr(void)
 {
        int i;
 
-       for (i = 0; i < ARRAY_SIZE(dynptr_tests); i++) {
-               if (!test__start_subtest(dynptr_tests[i].prog_name))
+       for (i = 0; i < ARRAY_SIZE(success_tests); i++) {
+               if (!test__start_subtest(success_tests[i]))
                        continue;
 
-               verify_success(dynptr_tests[i].prog_name);
+               verify_success(success_tests[i]);
        }
 
        RUN_TESTS(dynptr_fail);