Skip to content

Commit 5b46d23

Browse files
AsphalttKernel Patches Daemon
authored andcommitted
bpf: Add BPF_F_CPU and BPF_F_ALL_CPUS flags support for percpu_array maps
Introduce support for the BPF_F_ALL_CPUS flag in percpu_array maps to allow updating values for all CPUs with a single value for both update_elem and update_batch APIs. Introduce support for the BPF_F_CPU flag in percpu_array maps to allow: * update value for specified CPU for both update_elem and update_batch APIs. * lookup value for specified CPU for both lookup_elem and lookup_batch APIs. The BPF_F_CPU flag is passed via: * map_flags of lookup_elem and update_elem APIs along with embedded cpu info. * elem_flags of lookup_batch and update_batch APIs along with embedded cpu info. Signed-off-by: Leon Hwang <[email protected]>
1 parent dfa2ca7 commit 5b46d23

File tree

3 files changed

+29
-6
lines changed

3 files changed

+29
-6
lines changed

include/linux/bpf.h

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2698,7 +2698,7 @@ int map_set_for_each_callback_args(struct bpf_verifier_env *env,
26982698
struct bpf_func_state *callee);
26992699

27002700
int bpf_percpu_hash_copy(struct bpf_map *map, void *key, void *value);
2701-
int bpf_percpu_array_copy(struct bpf_map *map, void *key, void *value);
2701+
int bpf_percpu_array_copy(struct bpf_map *map, void *key, void *value, u64 flags);
27022702
int bpf_percpu_hash_update(struct bpf_map *map, void *key, void *value,
27032703
u64 flags);
27042704
int bpf_percpu_array_update(struct bpf_map *map, void *key, void *value,
@@ -3718,7 +3718,12 @@ struct bpf_prog *bpf_prog_find_from_stack(void);
37183718

37193719
static inline bool bpf_map_supports_cpu_flags(enum bpf_map_type map_type)
37203720
{
3721-
return false;
3721+
switch (map_type) {
3722+
case BPF_MAP_TYPE_PERCPU_ARRAY:
3723+
return true;
3724+
default:
3725+
return false;
3726+
}
37223727
}
37233728

37243729
static inline int bpf_map_check_op_flags(struct bpf_map *map, u64 flags, u64 allowed_flags)

kernel/bpf/arraymap.c

Lines changed: 21 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ static void *percpu_array_map_lookup_percpu_elem(struct bpf_map *map, void *key,
295295
return per_cpu_ptr(array->pptrs[index & array->index_mask], cpu);
296296
}
297297

298-
int bpf_percpu_array_copy(struct bpf_map *map, void *key, void *value)
298+
int bpf_percpu_array_copy(struct bpf_map *map, void *key, void *value, u64 map_flags)
299299
{
300300
struct bpf_array *array = container_of(map, struct bpf_array, map);
301301
u32 index = *(u32 *)key;
@@ -313,11 +313,18 @@ int bpf_percpu_array_copy(struct bpf_map *map, void *key, void *value)
313313
size = array->elem_size;
314314
rcu_read_lock();
315315
pptr = array->pptrs[index & array->index_mask];
316+
if (map_flags & BPF_F_CPU) {
317+
cpu = map_flags >> 32;
318+
copy_map_value_long(map, value, per_cpu_ptr(pptr, cpu));
319+
check_and_init_map_value(map, value);
320+
goto unlock;
321+
}
316322
for_each_possible_cpu(cpu) {
317323
copy_map_value_long(map, value + off, per_cpu_ptr(pptr, cpu));
318324
check_and_init_map_value(map, value + off);
319325
off += size;
320326
}
327+
unlock:
321328
rcu_read_unlock();
322329
return 0;
323330
}
@@ -390,7 +397,7 @@ int bpf_percpu_array_update(struct bpf_map *map, void *key, void *value,
390397
int cpu, off = 0;
391398
u32 size;
392399

393-
if (unlikely(map_flags > BPF_EXIST))
400+
if (unlikely((u32)map_flags > BPF_F_ALL_CPUS))
394401
/* unknown flags */
395402
return -EINVAL;
396403

@@ -411,11 +418,22 @@ int bpf_percpu_array_update(struct bpf_map *map, void *key, void *value,
411418
size = array->elem_size;
412419
rcu_read_lock();
413420
pptr = array->pptrs[index & array->index_mask];
421+
if (map_flags & BPF_F_CPU) {
422+
cpu = map_flags >> 32;
423+
copy_map_value_long(map, per_cpu_ptr(pptr, cpu), value);
424+
bpf_obj_free_fields(array->map.record, per_cpu_ptr(pptr, cpu));
425+
goto unlock;
426+
}
414427
for_each_possible_cpu(cpu) {
415428
copy_map_value_long(map, per_cpu_ptr(pptr, cpu), value + off);
416429
bpf_obj_free_fields(array->map.record, per_cpu_ptr(pptr, cpu));
417-
off += size;
430+
/* same user-provided value is used if BPF_F_ALL_CPUS is
431+
* specified, otherwise value is an array of per-CPU values.
432+
*/
433+
if (!(map_flags & BPF_F_ALL_CPUS))
434+
off += size;
418435
}
436+
unlock:
419437
rcu_read_unlock();
420438
return 0;
421439
}

kernel/bpf/syscall.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -316,7 +316,7 @@ static int bpf_map_copy_value(struct bpf_map *map, void *key, void *value,
316316
map->map_type == BPF_MAP_TYPE_LRU_PERCPU_HASH) {
317317
err = bpf_percpu_hash_copy(map, key, value);
318318
} else if (map->map_type == BPF_MAP_TYPE_PERCPU_ARRAY) {
319-
err = bpf_percpu_array_copy(map, key, value);
319+
err = bpf_percpu_array_copy(map, key, value, flags);
320320
} else if (map->map_type == BPF_MAP_TYPE_PERCPU_CGROUP_STORAGE) {
321321
err = bpf_percpu_cgroup_storage_copy(map, key, value);
322322
} else if (map->map_type == BPF_MAP_TYPE_STACK_TRACE) {

0 commit comments

Comments
 (0)