bpf: Remove redundant assignment to smap->map.value_size
authorYuntao Wang <ytcoode@gmail.com>
Wed, 23 Mar 2022 07:36:26 +0000 (15:36 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Fri, 1 Apr 2022 20:44:44 +0000 (22:44 +0200)
The attr->value_size is already assigned to smap->map.value_size
in bpf_map_init_from_attr(), there is no need to do it again in
stack_map_alloc().

Signed-off-by: Yuntao Wang <ytcoode@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Joanne Koong <joannelkoong@gmail.com>
Link: https://lore.kernel.org/bpf/20220323073626.958652-1-ytcoode@gmail.com
kernel/bpf/stackmap.c

index 34725bf..6131b4a 100644 (file)
@@ -106,7 +106,6 @@ static struct bpf_map *stack_map_alloc(union bpf_attr *attr)
                return ERR_PTR(-ENOMEM);
 
        bpf_map_init_from_attr(&smap->map, attr);
-       smap->map.value_size = value_size;
        smap->n_buckets = n_buckets;
 
        err = get_callchain_buffers(sysctl_perf_event_max_stack);