libbpf: Skip base btf sanity checks
authorAntoine Tenart <atenart@kernel.org>
Mon, 24 Jun 2024 09:09:07 +0000 (11:09 +0200)
committerAndrii Nakryiko <andrii@kernel.org>
Mon, 24 Jun 2024 16:19:42 +0000 (09:19 -0700)
commitc73a9683cb21012b6c0f14217974837151c527a8
treef5de59085d4b706dd8324c7167e9a9df7fefae03
parent5a532459aa919d055d822d8db4ea2c5c8d511568
libbpf: Skip base btf sanity checks

When upgrading to libbpf 1.3 we noticed a big performance hit while
loading programs using CORE on non base-BTF symbols. This was tracked
down to the new BTF sanity check logic. The issue is the base BTF
definitions are checked first for the base BTF and then again for every
module BTF.

Loading 5 dummy programs (using libbpf-rs) that are using CORE on a
non-base BTF symbol on my system:
- Before this fix: 3s.
- With this fix: 0.1s.

Fix this by only checking the types starting at the BTF start id. This
should ensure the base BTF is still checked as expected but only once
(btf->start_id == 1 when creating the base BTF), and then only
additional types are checked for each module BTF.

Fixes: 3903802bb99a ("libbpf: Add basic BTF sanity validation")
Signed-off-by: Antoine Tenart <atenart@kernel.org>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Reviewed-by: Alan Maguire <alan.maguire@oracle.com>
Link: https://lore.kernel.org/bpf/20240624090908.171231-1-atenart@kernel.org
tools/lib/bpf/btf.c