On Mon, Jun 29, 2020 at 5:35 PM Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx> wrote: > > From: Alexei Starovoitov <ast@xxxxxxxxxx> > > bpf_free_used_maps() or close(map_fd) will trigger map_free callback. > bpf_free_used_maps() is called after bpf prog is no longer executing: > bpf_prog_put->call_rcu->bpf_prog_free->bpf_free_used_maps. > Hence there is no need to call synchronize_rcu() to protect map elements. > > Signed-off-by: Alexei Starovoitov <ast@xxxxxxxxxx> > --- Seems correct. And nice that maps don't have to care about this anymore. Acked-by: Andrii Nakryiko <andriin@xxxxxx> > kernel/bpf/arraymap.c | 9 --------- > kernel/bpf/hashtab.c | 8 +++----- > kernel/bpf/lpm_trie.c | 5 ----- > kernel/bpf/queue_stack_maps.c | 7 ------- > kernel/bpf/reuseport_array.c | 2 -- > kernel/bpf/ringbuf.c | 7 ------- > kernel/bpf/stackmap.c | 3 --- > 7 files changed, 3 insertions(+), 38 deletions(-) >
