diff options
author | Hou Tao <houtao1@huawei.com> | 2023-12-08 18:23:49 +0800 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2023-12-09 18:12:33 -0800 |
commit | c26f2a8901393c9f81909da0a4324587092bd3a3 (patch) | |
tree | 50adfc5b3ca444245c15cc36ddc1e86ac01fade6 /kernel/bpf | |
parent | 32fa058398624166dd04ff4af49cfef69c94abbc (diff) | |
download | lwn-c26f2a8901393c9f81909da0a4324587092bd3a3.tar.gz lwn-c26f2a8901393c9f81909da0a4324587092bd3a3.zip |
bpf: Remove unnecessary wait from bpf_map_copy_value()
Both map_lookup_elem() and generic_map_lookup_batch() use
bpf_map_copy_value() to lookup and copy the value, and there is no
update operation in bpf_map_copy_value(), so just remove the invocation
of maybe_wait_bpf_programs() from it.
Fixes: 15c14a3dca42 ("bpf: Add bpf_map_{value_size, update_value, map_copy_value} functions")
Signed-off-by: Hou Tao <houtao1@huawei.com>
Link: https://lore.kernel.org/r/20231208102355.2628918-2-houtao@huaweicloud.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'kernel/bpf')
-rw-r--r-- | kernel/bpf/syscall.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c index aff045eed375..9ad3f527ab37 100644 --- a/kernel/bpf/syscall.c +++ b/kernel/bpf/syscall.c @@ -264,7 +264,6 @@ static int bpf_map_copy_value(struct bpf_map *map, void *key, void *value, } bpf_enable_instrumentation(); - maybe_wait_bpf_programs(map); return err; } |