On Mon, Nov 11, 2019 at 10:39 AM Jakub Kicinski <jakub.kicinski@xxxxxxxxxxxxx> wrote: > > On Sat, 9 Nov 2019 00:06:30 -0800, Andrii Nakryiko wrote: > > @@ -74,7 +78,7 @@ static struct bpf_map *array_map_alloc(union bpf_attr *attr) > > int ret, numa_node = bpf_map_attr_numa_node(attr); > > u32 elem_size, index_mask, max_entries; > > bool unpriv = !capable(CAP_SYS_ADMIN); > > - u64 cost, array_size, mask64; > > + u64 cost, array_size, data_size, mask64; > > struct bpf_map_memory mem; > > struct bpf_array *array; > > > > Please don't break reverse xmas tree where it exists. alright, will move it up