Re: [PATCH v5 0/5] BPF cleanup and tests

[Thomas Monjalon <thomas@monjalon.net>]

09/11/2025 21:07, Stephen Hemminger:
> This is a series of fixes and tests for BPF functionality.
> The new tests for Tx and Rx filtering require the infrastructure
> to build a bpf program (clang) and the tool to convert object
> file to header (xxd). If missing the test will get skipped.
> 
> It is disappointing that there were no examples or tests when
> the code was submitted.
> 
> v5 - review feedback
>      make it clear that load test is mirroring previous test
>      and check the result. 
> 
> v4 - review feedback - use alloca instead of burst to remove dependency on VLA
> 
> v3 - do not depend on xxd -n flag which is not present on older versions
> 
> Stephen Hemminger (5):
>   bpf: add allocation annotations to functions
>   bpf: use bulk free on filtered packets
>   bpf: add a test for BPF ELF load
>   bpf: add test for Rx and Tx filtering
>   bpf: replace use of VLA

I have these warnings:
app/test/test_bpf.c: duplicated include: rte_bpf.h
app/test/test_bpf.c: duplicated include: rte_ether.h
app/test/test_bpf.c: duplicated include: rte_ip.h

Given some issues are reported by Marat with the test,
I apply the patches which do not add new testing for now.

Please send a new version of the test when all is resolved.