Re: [PATCH v3 bpf-next 0/2] selftests/bpf: Test BTF sanitization

[Jiri Olsa <olsajiri@gmail.com>]

On Wed, Apr 08, 2026 at 05:57:33PM +0100, Alan Maguire wrote:
> Allow simulation of missing BPF features through provision of
> a synthetic feature cache set, and use this to simulate case
> where FEAT_BTF_LAYOUT is missing.  Ensure sanitization leaves us
> with expected BTF (layout info removed, layout header fields
> zeroed, strings data adjusted).
> 
> Specifying a feature cache with selected missing features will
> allow testing of other missing feature codepaths, but for now
> add BTF layout sanitization test only.
> 
> Changes since v2 [1]:
> 
> - change zfree() to free() since we immediately assign the
>   feat_cache (Jiri, patch 1)
> - "goto out" to avoid skeleton leak (Chengkaitao, patch 2)
> - just use kfree_skb__open() since we do not need to load
>   skeleton

Acked-by: Jiri Olsa <jolsa@kernel.org>

jirka

> 
> Changes since v1 [2]:
> 
> - renamed to bpf_object_set_feat_cache() (Andrii, patch 1)
> - remove __packed, relocate skeleton open/load, fix formatting
>   issues (Andrii, patch 2)
> 
> [1] https://lore.kernel.org/bpf/20260408105324.663280-1-alan.maguire@oracle.com/
> [2] https://lore.kernel.org/bpf/20260401164302.3844142-1-alan.maguire@oracle.com/
> 
> Alan Maguire (2):
>   libbpf: Allow use of feature cache for non-token cases
>   selftests/bpf: Add BTF sanitize test covering BTF layout
> 
>  tools/lib/bpf/libbpf.c                        | 12 ++-
>  tools/lib/bpf/libbpf_internal.h               |  3 +-
>  .../selftests/bpf/prog_tests/btf_sanitize.c   | 97 +++++++++++++++++++
>  3 files changed, 109 insertions(+), 3 deletions(-)
>  create mode 100644 tools/testing/selftests/bpf/prog_tests/btf_sanitize.c
> 
> -- 
> 2.39.3
>