]> git.baikalelectronics.ru Git - kernel.git/commit
libbpf: Don't require full struct enum64 in UAPI headers
authorAndrii Nakryiko <andrii@kernel.org>
Tue, 27 Sep 2022 04:29:39 +0000 (21:29 -0700)
committerDaniel Borkmann <daniel@iogearbox.net>
Tue, 27 Sep 2022 18:45:17 +0000 (20:45 +0200)
commit090714a09f3d14d867d88ceed6b9efdf3e24ee63
tree551e0f6feebbb075cd8ddd391058c806722b1ca7
parent440a4e61ec48c35800dcbe3e8280ec7685a81d12
libbpf: Don't require full struct enum64 in UAPI headers

Drop the requirement for system-wide kernel UAPI headers to provide full
struct btf_enum64 definition. This is an unexpected requirement that
slipped in libbpf 1.0 and put unnecessary pressure ([0]) on users to have
a bleeding-edge kernel UAPI header from unreleased Linux 6.0.

To achieve this, we forward declare struct btf_enum64. But that's not
enough as there is btf_enum64_value() helper that expects to know the
layout of struct btf_enum64. So we get a bit creative with
reinterpreting memory layout as array of __u32 and accesing lo32/hi32
fields as array elements. Alternative way would be to have a local
pointer variable for anonymous struct with exactly the same layout as
struct btf_enum64, but that gets us into C++ compiler errors complaining
about invalid type casts. So play it safe, if ugly.

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

Fixes: 5cfa9385526b ("libbpf: Add enum64 support for btf_dump")
Reported-by: Toke Høiland-Jørgensen <toke@toke.dk>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
Link: https://lore.kernel.org/bpf/20220927042940.147185-1-andrii@kernel.org
tools/lib/bpf/btf.h