diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2010-05-21 09:13:59 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-06-09 05:12:33 -0400 |
commit | 8ed92280be013180e24c84456ab6babcb07037cc (patch) | |
tree | 5b0826219599532cc1a17ed823bd2358a2bef666 /kernel/perf_event.c | |
parent | ecc55f84b2e9741f29daa787ded93986df6cbe17 (diff) |
perf, trace: Remove superfluous rcu_read_lock()
__DO_TRACE() already calls the callbacks under rcu_read_lock_sched(),
which is sufficient for our needs, avoid doing it again.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/perf_event.c')
-rw-r--r-- | kernel/perf_event.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index 4bd3b597bcca..b39bec346e80 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c | |||
@@ -4613,12 +4613,10 @@ void perf_tp_event(u64 addr, u64 count, void *record, int entry_size, | |||
4613 | perf_sample_data_init(&data, addr); | 4613 | perf_sample_data_init(&data, addr); |
4614 | data.raw = &raw; | 4614 | data.raw = &raw; |
4615 | 4615 | ||
4616 | rcu_read_lock(); | ||
4617 | hlist_for_each_entry_rcu(event, node, head, hlist_entry) { | 4616 | hlist_for_each_entry_rcu(event, node, head, hlist_entry) { |
4618 | if (perf_tp_event_match(event, &data, regs)) | 4617 | if (perf_tp_event_match(event, &data, regs)) |
4619 | perf_swevent_add(event, count, 1, &data, regs); | 4618 | perf_swevent_add(event, count, 1, &data, regs); |
4620 | } | 4619 | } |
4621 | rcu_read_unlock(); | ||
4622 | 4620 | ||
4623 | perf_swevent_put_recursion_context(rctx); | 4621 | perf_swevent_put_recursion_context(rctx); |
4624 | } | 4622 | } |