| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20190820114706.18546-4-toke@redhat.com>
Date: Tue, 20 Aug 2019 13:47:04 +0200
From: Toke Høiland-Jørgensen <toke@...hat.com>
To: Stephen Hemminger <stephen@...workplumber.org>,
Daniel Borkmann <daniel@...earbox.net>,
Alexei Starovoitov <ast@...nel.org>
Cc: Martin KaFai Lau <kafai@...com>, Song Liu <songliubraving@...com>,
Yonghong Song <yhs@...com>, David Miller <davem@...emloft.net>,
Jesper Dangaard Brouer <brouer@...hat.com>,
netdev@...r.kernel.org, bpf@...r.kernel.org,
Toke Høiland-Jørgensen <toke@...hat.com>
Subject: [RFC bpf-next 3/5] libbpf: Add support for specifying map pinning path via callback
This adds a callback parameter that can be set in struct bpf_prog_load_attr
which will allow the calling program to specify arbitrary paths for each
map included in an ELF file being loaded.
In particular, this allows iproute2 to implement its current semantics for
map pinning on top of libbpf.
Signed-off-by: Toke Høiland-Jørgensen <toke@...hat.com>
---
tools/lib/bpf/libbpf.c | 73 +++++++++++++++++++++++++++++-------------
tools/lib/bpf/libbpf.h | 6 ++++
2 files changed, 57 insertions(+), 22 deletions(-)
diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c
index 6d372a965c9d..a5c379378f26 100644
--- a/tools/lib/bpf/libbpf.c
+++ b/tools/lib/bpf/libbpf.c
@@ -3995,8 +3995,33 @@ int bpf_map__unpin(struct bpf_map *map, const char *path)
return 0;
}
-int bpf_object__pin_maps2(struct bpf_object *obj, const char *path,
- enum bpf_pin_mode mode)
+int __bpf_map__pin_path(const struct bpf_map *map,
+ const char *path,
+ map_pinning_path_fn fn, void *fn_ctx,
+ char *buf, int buf_len)
+{
+ const char *name = bpf_map__name(map);
+ int len;
+
+ if (fn) {
+ len = fn(fn_ctx, buf, buf_len, name, map->def.pinning);
+ buf[buf_len - 1] = '\0';
+ return len;
+ }
+
+ len = snprintf(buf, PATH_MAX, "%s/%s", path,
+ name);
+ if (len < 0)
+ return -EINVAL;
+ else if (len >= PATH_MAX)
+ return -ENAMETOOLONG;
+
+ return len;
+}
+
+int __bpf_object__pin_maps(struct bpf_object *obj, enum bpf_pin_mode mode,
+ const char *path,
+ map_pinning_path_fn cb, void *cb_ctx)
{
int explicit = (mode == BPF_PIN_MODE_EXPLICIT);
struct bpf_map *map;
@@ -4010,9 +4035,11 @@ int bpf_object__pin_maps2(struct bpf_object *obj, const char *path,
return -ENOENT;
}
- err = make_dir(path);
- if (err)
- return err;
+ if (path) {
+ err = make_dir(path);
+ if (err)
+ return err;
+ }
bpf_object__for_each_map(map, obj) {
char buf[PATH_MAX];
@@ -4021,14 +4048,10 @@ int bpf_object__pin_maps2(struct bpf_object *obj, const char *path,
if ((explicit && !map->def.pinning) || map->pin_reused)
continue;
- len = snprintf(buf, PATH_MAX, "%s/%s", path,
- bpf_map__name(map));
+ len = __bpf_map__pin_path(map, path, cb, cb_ctx, buf, PATH_MAX);
if (len < 0) {
err = -EINVAL;
goto err_unpin_maps;
- } else if (len >= PATH_MAX) {
- err = -ENAMETOOLONG;
- goto err_unpin_maps;
}
err = bpf_map__pin(map, buf);
@@ -4059,6 +4082,13 @@ int bpf_object__pin_maps2(struct bpf_object *obj, const char *path,
return err;
}
+int bpf_object__pin_maps2(struct bpf_object *obj, const char *path,
+ enum bpf_pin_mode mode)
+{
+ return __bpf_object__pin_maps(obj, mode, path, NULL, NULL);
+}
+
+
int bpf_object__pin_maps(struct bpf_object *obj, const char *path)
{
return bpf_object__pin_maps2(obj, path, BPF_PIN_MODE_ALL);
@@ -4914,23 +4944,21 @@ static int bpf_map_selfcheck_pinned(int fd, const struct bpf_map_def *map,
}
}
-
int bpf_probe_pinned(const struct bpf_map *map,
const struct bpf_prog_load_attr *attr)
{
const char *name = bpf_map__name(map);
char buf[PATH_MAX];
- int fd, len, ret;
+ int fd, ret;
- if (!attr->auto_pin_path)
+ if (!attr->auto_pin_path && !attr->auto_pin_cb)
return -ENOENT;
- len = snprintf(buf, PATH_MAX, "%s/%s", attr->auto_pin_path,
- name);
- if (len < 0)
- return -EINVAL;
- else if (len >= PATH_MAX)
- return -ENAMETOOLONG;
+ ret = __bpf_map__pin_path(map, attr->auto_pin_path,
+ attr->auto_pin_cb, attr->auto_pin_ctx,
+ buf, PATH_MAX);
+ if (ret < 0)
+ return ret;
fd = bpf_obj_get(buf);
if (fd <= 0)
@@ -5024,9 +5052,10 @@ int bpf_prog_load_xattr(const struct bpf_prog_load_attr *attr,
return -EINVAL;
}
- if (attr->auto_pin_path)
- bpf_object__pin_maps2(obj, attr->auto_pin_path,
- BPF_PIN_MODE_EXPLICIT);
+ if (attr->auto_pin_path || attr->auto_pin_cb)
+ __bpf_object__pin_maps(obj, BPF_PIN_MODE_EXPLICIT,
+ attr->auto_pin_path, attr->auto_pin_cb,
+ attr->auto_pin_ctx);
*pobj = obj;
*prog_fd = bpf_program__fd(first_prog);
diff --git a/tools/lib/bpf/libbpf.h b/tools/lib/bpf/libbpf.h
index 3c5c3256e22d..65fdd3389d27 100644
--- a/tools/lib/bpf/libbpf.h
+++ b/tools/lib/bpf/libbpf.h
@@ -72,6 +72,10 @@ enum bpf_pin_mode {
BPF_PIN_MODE_EXPLICIT,
};
+struct bpf_map_def;
+typedef int (*map_pinning_path_fn)(void *ctx, char *buf, int buf_len,
+ const char *name, unsigned int pinning);
+
LIBBPF_API struct bpf_object *bpf_object__open(const char *path);
LIBBPF_API struct bpf_object *
bpf_object__open_xattr(struct bpf_object_open_attr *attr);
@@ -361,6 +365,8 @@ struct bpf_prog_load_attr {
int log_level;
int prog_flags;
const char *auto_pin_path;
+ map_pinning_path_fn auto_pin_cb;
+ void *auto_pin_ctx;
};
LIBBPF_API int bpf_prog_load_xattr(const struct bpf_prog_load_attr *attr,
--
2.22.1
Powered by blists - more mailing lists