summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNamhyung Kim <namhyung.kim@lge.com>2012-05-31 14:51:44 +0900
committerArnaldo Carvalho de Melo <acme@redhat.com>2012-05-31 14:01:00 -0300
commitcb7225feec627e91d598198996429e9ee6804f8d (patch)
tree15b755aadc5c3df60282eddbddc4a542d829ddf8
parenta23c4dc422cff7bd30f40f5dc7c9ac4a6339005a (diff)
downloadtalos-obmc-linux-cb7225feec627e91d598198996429e9ee6804f8d.tar.gz
talos-obmc-linux-cb7225feec627e91d598198996429e9ee6804f8d.zip
perf: Remove duplicate invocation on perf_event_for_each
The @func callback was invoked twice for group leader when perf_event_for_each() called. It seems the commit 75f937f24bd9 ("perf_counter: Fix ctx->mutex vs counter ->mutex inversion") made the mistake during the change. Signed-off-by: Namhyung Kim <namhyung.kim@lge.com> Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Namhyung Kim <namhyung@gmail.com> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/1338443506-25009-1-git-send-email-namhyung.kim@lge.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r--kernel/events/core.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/events/core.c b/kernel/events/core.c
index 5b06cbbf6931..f85c0154b333 100644
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -3181,7 +3181,6 @@ static void perf_event_for_each(struct perf_event *event,
event = event->group_leader;
perf_event_for_each_child(event, func);
- func(event);
list_for_each_entry(sibling, &event->sibling_list, group_entry)
perf_event_for_each_child(sibling, func);
mutex_unlock(&ctx->mutex);
OpenPOWER on IntegriCloud