diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2011-02-22 10:52:24 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2011-02-23 05:35:47 -0500 |
commit | 768a06e2ca49cdf72389208cfc056a36cf8bc5e3 (patch) | |
tree | cd3e40f7d83526747785e7d4954ce96a5a3d69c7 /kernel/perf_event.c | |
parent | 3f7cce3c18188a067d463749168bdda5abc5b0f7 (diff) |
perf: Simplify task_clock_event_read()
There is no point in us having different code paths for nmi and !nmi
here, so remove the !nmi one.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.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 | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index dadeaea4b3fc..64a018e94fca 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c | |||
@@ -5810,16 +5810,9 @@ static void task_clock_event_del(struct perf_event *event, int flags) | |||
5810 | 5810 | ||
5811 | static void task_clock_event_read(struct perf_event *event) | 5811 | static void task_clock_event_read(struct perf_event *event) |
5812 | { | 5812 | { |
5813 | u64 time; | 5813 | u64 now = perf_clock(); |
5814 | 5814 | u64 delta = now - event->ctx->timestamp; | |
5815 | if (!in_nmi()) { | 5815 | u64 time = event->ctx->time + delta; |
5816 | update_context_time(event->ctx); | ||
5817 | time = event->ctx->time; | ||
5818 | } else { | ||
5819 | u64 now = perf_clock(); | ||
5820 | u64 delta = now - event->ctx->timestamp; | ||
5821 | time = event->ctx->time + delta; | ||
5822 | } | ||
5823 | 5816 | ||
5824 | task_clock_event_update(event, time); | 5817 | task_clock_event_update(event, time); |
5825 | } | 5818 | } |