mirror of
https://github.com/netdata/libbpf.git
synced 2026-03-26 19:29:07 +08:00
libbpf: Add btf__set_fd() for more control over loaded BTF FD
Add setter for BTF FD to allow application more fine-grained control in more advanced scenarios. Storing BTF FD inside `struct btf` provides little benefit and probably would be better done differently (e.g., btf__load() could just return FD on success), but we are stuck with this due to backwards compatibility. The main problem is that it's impossible to load BTF and than free user-space memory, but keep FD intact, because `struct btf` assumes ownership of that FD upon successful load and will attempt to close it during btf__free(). To allow callers (e.g., libbpf itself for BTF sanitization) to have more control over this, add btf__set_fd() to allow to reset FD arbitrarily, if necessary. Signed-off-by: Andrii Nakryiko <andriin@fb.com> Signed-off-by: Daniel Borkmann <daniel@iogearbox.net> Link: https://lore.kernel.org/bpf/20200708015318.3827358-3-andriin@fb.com
This commit is contained in:
committed by
Andrii Nakryiko
parent
133543c202
commit
d5a36e2070
@@ -389,7 +389,7 @@ void btf__free(struct btf *btf)
|
||||
if (!btf)
|
||||
return;
|
||||
|
||||
if (btf->fd != -1)
|
||||
if (btf->fd >= 0)
|
||||
close(btf->fd);
|
||||
|
||||
free(btf->data);
|
||||
@@ -700,6 +700,11 @@ int btf__fd(const struct btf *btf)
|
||||
return btf->fd;
|
||||
}
|
||||
|
||||
void btf__set_fd(struct btf *btf, int fd)
|
||||
{
|
||||
btf->fd = fd;
|
||||
}
|
||||
|
||||
const void *btf__get_raw_data(const struct btf *btf, __u32 *size)
|
||||
{
|
||||
*size = btf->data_size;
|
||||
|
||||
Reference in New Issue
Block a user