diff options
author | Yonghong Song <yhs@fb.com> | 2017-10-30 13:50:22 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-11-01 12:35:48 +0900 |
commit | 07c41a295c5f25928a7cb689fdec816bd0089fe8 (patch) | |
tree | 069bf405b725d169ce806019e0d4a612d31b456c /drivers | |
parent | 3051fbec206eb6967b7fdecedb63ebb1ed67a1a7 (diff) | |
download | talos-obmc-linux-07c41a295c5f25928a7cb689fdec816bd0089fe8.tar.gz talos-obmc-linux-07c41a295c5f25928a7cb689fdec816bd0089fe8.zip |
bpf: avoid rcu_dereference inside bpf_event_mutex lock region
During perf event attaching/detaching bpf programs,
the tp_event->prog_array change is protected by the
bpf_event_mutex lock in both attaching and deteching
functions. Although tp_event->prog_array is a rcu
pointer, rcu_derefrence is not needed to access it
since mutex lock will guarantee ordering.
Verified through "make C=2" that sparse
locking check still happy with the new change.
Also change the label name in perf_event_{attach,detach}_bpf_prog
from "out" to "unlock" to reflect the code action after the label.
Signed-off-by: Yonghong Song <yhs@fb.com>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: Martin KaFai Lau <kafai@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
0 files changed, 0 insertions, 0 deletions