On Thu, Aug 21, 2025 at 5:49 AM Andrii Nakryiko
<andrii.nakryiko@xxxxxxxxx> wrote:
>
> On Sun, Aug 17, 2025 at 11:31 PM Hengqi Chen <hengqi.chen@xxxxxxxxx> wrote:
> >
> > Some of the bpf test progs still use linux/libc headers.
> > Let's use vmlinux.h instead like the rest of test progs.
> > This will also ease cross compiling.
> >
> > Signed-off-by: Hengqi Chen <hengqi.chen@xxxxxxxxx>
> > ---
> > tools/testing/selftests/bpf/progs/loop1.c | 7 +------
> > tools/testing/selftests/bpf/progs/loop2.c | 7 +------
> > tools/testing/selftests/bpf/progs/loop3.c | 7 +------
> > tools/testing/selftests/bpf/progs/loop6.c | 15 ++++-----------
> > tools/testing/selftests/bpf/progs/test_overhead.c | 5 +----
> > 5 files changed, 8 insertions(+), 33 deletions(-)
> >
>
> I'm getting this:
>
>
> progs/loop6.c:58:5: error: redefinition of 'config' as different kind of symbol
> 58 | int config = 0;
> | CLNG-BPF [test_progs] sock_addr_kern.bpf.o
> ^
> /data/users/andriin/linux/tools/testing/selftests/bpf/tools/include/vmlinux.h:71906:25:
> note: previous definition is here
> 71906 | typedef struct config_s config;
> | ^
> progs/loop6.c:69:6: error: unexpected type name 'config': expected expression
> 69 | if (config != 0)
> | ^
> CLNG-BPF [test_progs] sock_destroy_prog.bpf.o
> progs/loop6.c:92:9: error: expected identifier or '('
> 92 | config = 1;
> | ^
>
>
> So let's rename a too generically named variable?
>
OK. Will rename to `run_once` based on the context as AI suggested.
> pw-bot: cr
>
>
> [...]
