]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
libbpf: deprecate legacy BPF map definitions
authorAndrii Nakryiko <andrii@kernel.org>
Thu, 20 Jan 2022 06:05:28 +0000 (22:05 -0800)
committerAlexei Starovoitov <ast@kernel.org>
Fri, 21 Jan 2022 05:19:05 +0000 (21:19 -0800)
commit93b8952d223af03c51fba0c6258173d2ffbd2cb7
tree65fc57206b4dd338b08d2a9bfc55688536a503e9
parentccc3f56918f670ccb5b8df2c6a15cbd083a4dd03
libbpf: deprecate legacy BPF map definitions

Enact deprecation of legacy BPF map definition in SEC("maps") ([0]). For
the definitions themselves introduce LIBBPF_STRICT_MAP_DEFINITIONS flag
for libbpf strict mode. If it is set, error out on any struct
bpf_map_def-based map definition. If not set, libbpf will print out
a warning for each legacy BPF map to raise awareness that it goes away.

For any use of BPF_ANNOTATE_KV_PAIR() macro providing a legacy way to
associate BTF key/value type information with legacy BPF map definition,
warn through libbpf's pr_warn() error message (but don't fail BPF object
open).

BPF-side struct bpf_map_def is marked as deprecated. User-space struct
bpf_map_def has to be used internally in libbpf, so it is left
untouched. It should be enough for bpf_map__def() to be marked
deprecated to raise awareness that it goes away.

bpftool is an interesting case that utilizes libbpf to open BPF ELF
object to generate skeleton. As such, even though bpftool itself uses
full on strict libbpf mode (LIBBPF_STRICT_ALL), it has to relax it a bit
for BPF map definition handling to minimize unnecessary disruptions. So
opt-out of LIBBPF_STRICT_MAP_DEFINITIONS for bpftool. User's code that
will later use generated skeleton will make its own decision whether to
enforce LIBBPF_STRICT_MAP_DEFINITIONS or not.

There are few tests in selftests/bpf that are consciously using legacy
BPF map definitions to test libbpf functionality. For those, temporary
opt out of LIBBPF_STRICT_MAP_DEFINITIONS mode for the duration of those
tests.

  [0] Closes: https://github.com/libbpf/libbpf/issues/272

Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/r/20220120060529.1890907-4-andrii@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/bpf/bpftool/main.c
tools/lib/bpf/bpf_helpers.h
tools/lib/bpf/libbpf.c
tools/lib/bpf/libbpf_legacy.h
tools/testing/selftests/bpf/prog_tests/btf.c