selftests/bpf: Add type-match checks to type-based tests

Now that we have type-match logic in both libbpf and the kernel, this
change adjusts the existing BPF self tests to check this functionality.
Specifically, we extend the existing type-based tests to check the
previously introduced bpf_core_type_matches macro.

Signed-off-by: Daniel Müller <deso@posteo.net>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20220628160127.607834-8-deso@posteo.net

diff --git a/tools/testing/selftests/bpf/prog_tests/core_reloc.c b/tools/testing/selftests/bpf/prog_tests/core_reloc.c
index 2f92feb809be8c4a1f4392f7908ede5055a0aca1..328dd744e5bd6715f6945625c2d9c72dea2e937b 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/core_reloc.c
+++ b/tools/testing/selftests/bpf/prog_tests/core_reloc.c
@@ -543,7 +543,6 @@ static int __trigger_module_test_read(const struct core_reloc_test_case *test)
        return 0;
 }
 
-
 static const struct core_reloc_test_case test_cases[] = {
        /* validate we can find kernel image and use its BTF for relocs */
        {
@@ -752,7 +751,7 @@ static const struct core_reloc_test_case test_cases[] = {
        SIZE_CASE(size___diff_offs),
        SIZE_ERR_CASE(size___err_ambiguous),
 
-       /* validate type existence and size relocations */
+       /* validate type existence, match, and size relocations */
        TYPE_BASED_CASE(type_based, {
                .struct_exists = 1,
                .union_exists = 1,
@@ -765,6 +764,19 @@ static const struct core_reloc_test_case test_cases[] = {
                .typedef_void_ptr_exists = 1,
                .typedef_func_proto_exists = 1,
                .typedef_arr_exists = 1,
+
+               .struct_matches = 1,
+               .union_matches = 1,
+               .enum_matches = 1,
+               .typedef_named_struct_matches = 1,
+               .typedef_anon_struct_matches = 1,
+               .typedef_struct_ptr_matches = 1,
+               .typedef_int_matches = 1,
+               .typedef_enum_matches = 1,
+               .typedef_void_ptr_matches = 1,
+               .typedef_func_proto_matches = 1,
+               .typedef_arr_matches = 1,
+
                .struct_sz = sizeof(struct a_struct),
                .union_sz = sizeof(union a_union),
                .enum_sz = sizeof(enum an_enum),
@@ -792,6 +804,19 @@ static const struct core_reloc_test_case test_cases[] = {
                .typedef_void_ptr_exists = 1,
                .typedef_func_proto_exists = 1,
                .typedef_arr_exists = 1,
+
+               .struct_matches = 0,
+               .union_matches = 0,
+               .enum_matches = 0,
+               .typedef_named_struct_matches = 0,
+               .typedef_anon_struct_matches = 0,
+               .typedef_struct_ptr_matches = 1,
+               .typedef_int_matches = 0,
+               .typedef_enum_matches = 0,
+               .typedef_void_ptr_matches = 1,
+               .typedef_func_proto_matches = 0,
+               .typedef_arr_matches = 0,
+
                .struct_sz = sizeof(struct a_struct___diff_sz),
                .union_sz = sizeof(union a_union___diff_sz),
                .enum_sz = sizeof(enum an_enum___diff_sz),
@@ -806,10 +831,12 @@ static const struct core_reloc_test_case test_cases[] = {
        }),
        TYPE_BASED_CASE(type_based___incompat, {
                .enum_exists = 1,
+               .enum_matches = 1,
                .enum_sz = sizeof(enum an_enum),
        }),
        TYPE_BASED_CASE(type_based___fn_wrong_args, {
                .struct_exists = 1,
+               .struct_matches = 1,
                .struct_sz = sizeof(struct a_struct),
        }),
 

diff --git a/tools/testing/selftests/bpf/progs/core_reloc_types.h b/tools/testing/selftests/bpf/progs/core_reloc_types.h
index 26e103302c05156e86922f49cc2f7f299d42b9ea..6a44f3e63ae530b2dfc3174068467e4bd818645e 100644 (file)
--- a/tools/testing/selftests/bpf/progs/core_reloc_types.h
+++ b/tools/testing/selftests/bpf/progs/core_reloc_types.h
@@ -860,7 +860,7 @@ struct core_reloc_size___err_ambiguous2 {
 };
 
 /*
- * TYPE EXISTENCE & SIZE
+ * TYPE EXISTENCE, MATCH & SIZE
  */
 struct core_reloc_type_based_output {
        bool struct_exists;
@@ -875,6 +875,18 @@ struct core_reloc_type_based_output {
        bool typedef_func_proto_exists;
        bool typedef_arr_exists;
 
+       bool struct_matches;
+       bool union_matches;
+       bool enum_matches;
+       bool typedef_named_struct_matches;
+       bool typedef_anon_struct_matches;
+       bool typedef_struct_ptr_matches;
+       bool typedef_int_matches;
+       bool typedef_enum_matches;
+       bool typedef_void_ptr_matches;
+       bool typedef_func_proto_matches;
+       bool typedef_arr_matches;
+
        int struct_sz;
        int union_sz;
        int enum_sz;

diff --git a/tools/testing/selftests/bpf/progs/test_core_reloc_type_based.c b/tools/testing/selftests/bpf/progs/test_core_reloc_type_based.c
index fb60f8195c53d8ade6000adb40ac4b69e3273106..325ead66613011ac06044d2604967d539727e547 100644 (file)
--- a/tools/testing/selftests/bpf/progs/test_core_reloc_type_based.c
+++ b/tools/testing/selftests/bpf/progs/test_core_reloc_type_based.c
@@ -61,6 +61,18 @@ struct core_reloc_type_based_output {
        bool typedef_func_proto_exists;
        bool typedef_arr_exists;
 
+       bool struct_matches;
+       bool union_matches;
+       bool enum_matches;
+       bool typedef_named_struct_matches;
+       bool typedef_anon_struct_matches;
+       bool typedef_struct_ptr_matches;
+       bool typedef_int_matches;
+       bool typedef_enum_matches;
+       bool typedef_void_ptr_matches;
+       bool typedef_func_proto_matches;
+       bool typedef_arr_matches;
+
        int struct_sz;
        int union_sz;
        int enum_sz;
@@ -77,7 +89,13 @@ struct core_reloc_type_based_output {
 SEC("raw_tracepoint/sys_enter")
 int test_core_type_based(void *ctx)
 {
-#if __has_builtin(__builtin_preserve_type_info)
+       /* Support for the BPF_TYPE_MATCHES argument to the
+        * __builtin_preserve_type_info builtin was added at some point during
+        * development of clang 15 and it's what we require for this test. Part of it
+        * could run with merely __builtin_preserve_type_info (which could be checked
+        * separately), but we have to find an upper bound.
+        */
+#if __has_builtin(__builtin_preserve_type_info) && __clang_major__ >= 15
        struct core_reloc_type_based_output *out = (void *)&data.out;
 
        out->struct_exists = bpf_core_type_exists(struct a_struct);
@@ -92,6 +110,18 @@ int test_core_type_based(void *ctx)
        out->typedef_func_proto_exists = bpf_core_type_exists(func_proto_typedef);
        out->typedef_arr_exists = bpf_core_type_exists(arr_typedef);
 
+       out->struct_matches = bpf_core_type_matches(struct a_struct);
+       out->union_matches = bpf_core_type_matches(union a_union);
+       out->enum_matches = bpf_core_type_matches(enum an_enum);
+       out->typedef_named_struct_matches = bpf_core_type_matches(named_struct_typedef);
+       out->typedef_anon_struct_matches = bpf_core_type_matches(anon_struct_typedef);
+       out->typedef_struct_ptr_matches = bpf_core_type_matches(struct_ptr_typedef);
+       out->typedef_int_matches = bpf_core_type_matches(int_typedef);
+       out->typedef_enum_matches = bpf_core_type_matches(enum_typedef);
+       out->typedef_void_ptr_matches = bpf_core_type_matches(void_ptr_typedef);
+       out->typedef_func_proto_matches = bpf_core_type_matches(func_proto_typedef);
+       out->typedef_arr_matches = bpf_core_type_matches(arr_typedef);
+
        out->struct_sz = bpf_core_type_size(struct a_struct);
        out->union_sz = bpf_core_type_size(union a_union);
        out->enum_sz = bpf_core_type_size(enum an_enum);