Commit 33a2c75c authored by Stanislav Fomichev's avatar Stanislav Fomichev Committed by Alexei Starovoitov

libbpf: add internal pin_name

pin_name is the same as section_name where '/' is replaced
by '_'. bpf_object__pin_programs is converted to use pin_name
to avoid the situation where section_name would require creating another
subdirectory for a pin (as, for example, when calling bpf_object__pin_programs
for programs in sections like "cgroup/connect6").
Signed-off-by: default avatarStanislav Fomichev <sdf@google.com>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
parent fd734c5c
...@@ -124,6 +124,10 @@ struct bpf_program { ...@@ -124,6 +124,10 @@ struct bpf_program {
char *name; char *name;
int prog_ifindex; int prog_ifindex;
char *section_name; char *section_name;
/* section_name with / replaced by _; makes recursive pinning
* in bpf_object__pin_programs easier
*/
char *pin_name;
struct bpf_insn *insns; struct bpf_insn *insns;
size_t insns_cnt, main_prog_cnt; size_t insns_cnt, main_prog_cnt;
enum bpf_prog_type type; enum bpf_prog_type type;
...@@ -253,6 +257,7 @@ static void bpf_program__exit(struct bpf_program *prog) ...@@ -253,6 +257,7 @@ static void bpf_program__exit(struct bpf_program *prog)
bpf_program__unload(prog); bpf_program__unload(prog);
zfree(&prog->name); zfree(&prog->name);
zfree(&prog->section_name); zfree(&prog->section_name);
zfree(&prog->pin_name);
zfree(&prog->insns); zfree(&prog->insns);
zfree(&prog->reloc_desc); zfree(&prog->reloc_desc);
...@@ -261,6 +266,17 @@ static void bpf_program__exit(struct bpf_program *prog) ...@@ -261,6 +266,17 @@ static void bpf_program__exit(struct bpf_program *prog)
prog->idx = -1; prog->idx = -1;
} }
static char *__bpf_program__pin_name(struct bpf_program *prog)
{
char *name, *p;
name = p = strdup(prog->section_name);
while ((p = strchr(p, '/')))
*p = '_';
return name;
}
static int static int
bpf_program__init(void *data, size_t size, char *section_name, int idx, bpf_program__init(void *data, size_t size, char *section_name, int idx,
struct bpf_program *prog) struct bpf_program *prog)
...@@ -279,6 +295,13 @@ bpf_program__init(void *data, size_t size, char *section_name, int idx, ...@@ -279,6 +295,13 @@ bpf_program__init(void *data, size_t size, char *section_name, int idx,
goto errout; goto errout;
} }
prog->pin_name = __bpf_program__pin_name(prog);
if (!prog->pin_name) {
pr_warning("failed to alloc pin name for prog under section(%d) %s\n",
idx, section_name);
goto errout;
}
prog->insns = malloc(size); prog->insns = malloc(size);
if (!prog->insns) { if (!prog->insns) {
pr_warning("failed to alloc insns for prog under section %s\n", pr_warning("failed to alloc insns for prog under section %s\n",
...@@ -2006,7 +2029,7 @@ int bpf_object__pin_programs(struct bpf_object *obj, const char *path) ...@@ -2006,7 +2029,7 @@ int bpf_object__pin_programs(struct bpf_object *obj, const char *path)
int len; int len;
len = snprintf(buf, PATH_MAX, "%s/%s", path, len = snprintf(buf, PATH_MAX, "%s/%s", path,
prog->section_name); prog->pin_name);
if (len < 0) { if (len < 0) {
err = -EINVAL; err = -EINVAL;
goto err_unpin_programs; goto err_unpin_programs;
...@@ -2028,7 +2051,7 @@ int bpf_object__pin_programs(struct bpf_object *obj, const char *path) ...@@ -2028,7 +2051,7 @@ int bpf_object__pin_programs(struct bpf_object *obj, const char *path)
int len; int len;
len = snprintf(buf, PATH_MAX, "%s/%s", path, len = snprintf(buf, PATH_MAX, "%s/%s", path,
prog->section_name); prog->pin_name);
if (len < 0) if (len < 0)
continue; continue;
else if (len >= PATH_MAX) else if (len >= PATH_MAX)
...@@ -2053,7 +2076,7 @@ int bpf_object__unpin_programs(struct bpf_object *obj, const char *path) ...@@ -2053,7 +2076,7 @@ int bpf_object__unpin_programs(struct bpf_object *obj, const char *path)
int len; int len;
len = snprintf(buf, PATH_MAX, "%s/%s", path, len = snprintf(buf, PATH_MAX, "%s/%s", path,
prog->section_name); prog->pin_name);
if (len < 0) if (len < 0)
return -EINVAL; return -EINVAL;
else if (len >= PATH_MAX) else if (len >= PATH_MAX)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment