libbpf: Expose BPF program's function name
Add APIs to get BPF program function name, as opposed to bpf_program__title(), which returns BPF program function's section name. Function name has a benefit of being a valid C identifier and uniquely identifies a specific BPF program, while section name can be duplicated across multiple independent BPF programs. Add also bpf_object__find_program_by_name(), similar to bpf_object__find_program_by_title(), to facilitate looking up BPF programs by their C function names. Convert one of selftests to new API for look up. Signed-off-by: Andrii Nakryiko <andriin@fb.com> Signed-off-by: Alexei Starovoitov <ast@kernel.org> Acked-by: Martin KaFai Lau <kafai@fb.com> Link: https://lore.kernel.org/bpf/20191214014341.3442258-9-andriin@fb.com
This commit is contained in:

committed by
Alexei Starovoitov

parent
9f81654eeb
commit
01af3bf067
@@ -209,8 +209,8 @@ static const char * const libbpf_type_to_btf_name[] = {
|
||||
};
|
||||
|
||||
struct bpf_map {
|
||||
int fd;
|
||||
char *name;
|
||||
int fd;
|
||||
int sec_idx;
|
||||
size_t sec_offset;
|
||||
int map_ifindex;
|
||||
@@ -1384,7 +1384,7 @@ static int bpf_object__init_user_btf_maps(struct bpf_object *obj, bool strict,
|
||||
}
|
||||
|
||||
static int bpf_object__init_maps(struct bpf_object *obj,
|
||||
struct bpf_object_open_opts *opts)
|
||||
const struct bpf_object_open_opts *opts)
|
||||
{
|
||||
const char *pin_root_path = OPTS_GET(opts, pin_root_path, NULL);
|
||||
bool strict = !OPTS_GET(opts, relaxed_maps, false);
|
||||
@@ -1748,6 +1748,19 @@ bpf_object__find_program_by_title(const struct bpf_object *obj,
|
||||
return NULL;
|
||||
}
|
||||
|
||||
struct bpf_program *
|
||||
bpf_object__find_program_by_name(const struct bpf_object *obj,
|
||||
const char *name)
|
||||
{
|
||||
struct bpf_program *prog;
|
||||
|
||||
bpf_object__for_each_program(prog, obj) {
|
||||
if (!strcmp(prog->name, name))
|
||||
return prog;
|
||||
}
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static bool bpf_object__shndx_is_data(const struct bpf_object *obj,
|
||||
int shndx)
|
||||
{
|
||||
@@ -3894,7 +3907,7 @@ static int libbpf_find_attach_btf_id(const char *name,
|
||||
__u32 attach_prog_fd);
|
||||
static struct bpf_object *
|
||||
__bpf_object__open(const char *path, const void *obj_buf, size_t obj_buf_sz,
|
||||
struct bpf_object_open_opts *opts)
|
||||
const struct bpf_object_open_opts *opts)
|
||||
{
|
||||
struct bpf_program *prog;
|
||||
struct bpf_object *obj;
|
||||
@@ -4003,7 +4016,7 @@ struct bpf_object *bpf_object__open(const char *path)
|
||||
}
|
||||
|
||||
struct bpf_object *
|
||||
bpf_object__open_file(const char *path, struct bpf_object_open_opts *opts)
|
||||
bpf_object__open_file(const char *path, const struct bpf_object_open_opts *opts)
|
||||
{
|
||||
if (!path)
|
||||
return ERR_PTR(-EINVAL);
|
||||
@@ -4015,7 +4028,7 @@ bpf_object__open_file(const char *path, struct bpf_object_open_opts *opts)
|
||||
|
||||
struct bpf_object *
|
||||
bpf_object__open_mem(const void *obj_buf, size_t obj_buf_sz,
|
||||
struct bpf_object_open_opts *opts)
|
||||
const struct bpf_object_open_opts *opts)
|
||||
{
|
||||
if (!obj_buf || obj_buf_sz == 0)
|
||||
return ERR_PTR(-EINVAL);
|
||||
@@ -4820,6 +4833,11 @@ void bpf_program__set_ifindex(struct bpf_program *prog, __u32 ifindex)
|
||||
prog->prog_ifindex = ifindex;
|
||||
}
|
||||
|
||||
const char *bpf_program__name(const struct bpf_program *prog)
|
||||
{
|
||||
return prog->name;
|
||||
}
|
||||
|
||||
const char *bpf_program__title(const struct bpf_program *prog, bool needs_copy)
|
||||
{
|
||||
const char *title;
|
||||
|
Reference in New Issue
Block a user