diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-11-24 14:38:22 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-11-24 15:34:00 -0500 |
commit | fe6126722718e51fba4879517c11ac12d9775bcc (patch) | |
tree | 12a2c8f0f39fd1051dfa0a1132f5c07f9a733362 | |
parent | 1261a02a0c0ab8e643125705f0d1d83e5090e4d1 (diff) |
perf_events: Fix bad software/trace event recursion counting
Commit 4ed7c92d68a5387ba5f7030dc76eab03558e27f5
(perf_events: Undo some recursion damage) has introduced a bad
reference counting of the recursion context. putting the context
behaves like getting it, dropping every software/trace events
after the first one in a context.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Arjan van de Ven <arjan@infradead.org>
Cc: Li Zefan <lizf@cn.fujitsu.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
LKML-Reference: <1259091502-5171-1-git-send-email-fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/perf_event.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index accfd7bfe387..35df94e344f2 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c | |||
@@ -3914,7 +3914,7 @@ void perf_swevent_put_recursion_context(int rctx) | |||
3914 | { | 3914 | { |
3915 | struct perf_cpu_context *cpuctx = &__get_cpu_var(perf_cpu_context); | 3915 | struct perf_cpu_context *cpuctx = &__get_cpu_var(perf_cpu_context); |
3916 | barrier(); | 3916 | barrier(); |
3917 | cpuctx->recursion[rctx]++; | 3917 | cpuctx->recursion[rctx]--; |
3918 | put_cpu_var(perf_cpu_context); | 3918 | put_cpu_var(perf_cpu_context); |
3919 | } | 3919 | } |
3920 | EXPORT_SYMBOL_GPL(perf_swevent_put_recursion_context); | 3920 | EXPORT_SYMBOL_GPL(perf_swevent_put_recursion_context); |