diff options
author | Roman Gushchin <guro@fb.com> | 2018-09-28 10:45:40 -0400 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2018-10-01 10:18:32 -0400 |
commit | f294b37ec7b24a574884cd157497a3748081c0f0 (patch) | |
tree | fb5bdf6cf4a5ff61b7a5f84274ae82c123aac7a9 /include/linux/bpf-cgroup.h | |
parent | 8bad74f9840f87661f20ced3dc80c84ab4fd55a1 (diff) |
bpf: rework cgroup storage pointer passing
To simplify the following introduction of per-cpu cgroup storage,
let's rework a bit a mechanism of passing a pointer to a cgroup
storage into the bpf_get_local_storage(). Let's save a pointer
to the corresponding bpf_cgroup_storage structure, instead of
a pointer to the actual buffer.
It will help us to handle per-cpu storage later, which has
a different way of accessing to the actual data.
Signed-off-by: Roman Gushchin <guro@fb.com>
Acked-by: Song Liu <songliubraving@fb.com>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Diffstat (limited to 'include/linux/bpf-cgroup.h')
-rw-r--r-- | include/linux/bpf-cgroup.h | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/include/linux/bpf-cgroup.h b/include/linux/bpf-cgroup.h index e9871b012dac..7e0c9a1d48b7 100644 --- a/include/linux/bpf-cgroup.h +++ b/include/linux/bpf-cgroup.h | |||
@@ -23,7 +23,8 @@ struct bpf_cgroup_storage; | |||
23 | extern struct static_key_false cgroup_bpf_enabled_key; | 23 | extern struct static_key_false cgroup_bpf_enabled_key; |
24 | #define cgroup_bpf_enabled static_branch_unlikely(&cgroup_bpf_enabled_key) | 24 | #define cgroup_bpf_enabled static_branch_unlikely(&cgroup_bpf_enabled_key) |
25 | 25 | ||
26 | DECLARE_PER_CPU(void*, bpf_cgroup_storage[MAX_BPF_CGROUP_STORAGE_TYPE]); | 26 | DECLARE_PER_CPU(struct bpf_cgroup_storage*, |
27 | bpf_cgroup_storage[MAX_BPF_CGROUP_STORAGE_TYPE]); | ||
27 | 28 | ||
28 | #define for_each_cgroup_storage_type(stype) \ | 29 | #define for_each_cgroup_storage_type(stype) \ |
29 | for (stype = 0; stype < MAX_BPF_CGROUP_STORAGE_TYPE; stype++) | 30 | for (stype = 0; stype < MAX_BPF_CGROUP_STORAGE_TYPE; stype++) |
@@ -115,15 +116,9 @@ static inline void bpf_cgroup_storage_set(struct bpf_cgroup_storage | |||
115 | *storage[MAX_BPF_CGROUP_STORAGE_TYPE]) | 116 | *storage[MAX_BPF_CGROUP_STORAGE_TYPE]) |
116 | { | 117 | { |
117 | enum bpf_cgroup_storage_type stype; | 118 | enum bpf_cgroup_storage_type stype; |
118 | struct bpf_storage_buffer *buf; | ||
119 | |||
120 | for_each_cgroup_storage_type(stype) { | ||
121 | if (!storage[stype]) | ||
122 | continue; | ||
123 | 119 | ||
124 | buf = READ_ONCE(storage[stype]->buf); | 120 | for_each_cgroup_storage_type(stype) |
125 | this_cpu_write(bpf_cgroup_storage[stype], &buf->data[0]); | 121 | this_cpu_write(bpf_cgroup_storage[stype], storage[stype]); |
126 | } | ||
127 | } | 122 | } |
128 | 123 | ||
129 | struct bpf_cgroup_storage *bpf_cgroup_storage_alloc(struct bpf_prog *prog, | 124 | struct bpf_cgroup_storage *bpf_cgroup_storage_alloc(struct bpf_prog *prog, |