summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Zijlstra <a.p.zijlstra@chello.nl>2009-11-20 22:19:53 +0100
committerIngo Molnar <mingo@elte.hu>2009-11-21 14:11:39 +0100
commit58e5ad1de3d6ad931c84f0cc8ef0655c922f30ad (patch)
treef3b018f0f8ceaf21747b5e1d912ad30b40d00779
parent3dbebf15c5d3e265f751eec72c1538a00da4be27 (diff)
downloadlinux-58e5ad1de3d6ad931c84f0cc8ef0655c922f30ad.tar.bz2
perf: Simplify __perf_event_read
cpuctx is always active, task context is always active for current the previous condition verifies that if its a task context its for current, hence we can assume ctx->is_active. Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Paul Mackerras <paulus@samba.org> LKML-Reference: <20091120212509.000272254@chello.nl> Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r--kernel/perf_event.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c
index af150bbcfc5b..028619dd6d0e 100644
--- a/kernel/perf_event.c
+++ b/kernel/perf_event.c
@@ -1526,10 +1526,9 @@ static void __perf_event_read(void *info)
if (ctx->task && cpuctx->task_ctx != ctx)
return;
- if (ctx->is_active)
- update_context_time(ctx);
- event->pmu->read(event);
+ update_context_time(ctx);
update_event_times(event);
+ event->pmu->read(event);
}
static u64 perf_event_read(struct perf_event *event)