diff options
author | Steven Rostedt <rostedt@goodmis.org> | 2008-10-02 19:23:04 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-10-14 04:39:16 -0400 |
commit | 6450c1d3213e27b0dcbf34cce7ad1ae74244c520 (patch) | |
tree | b8b51924fcbe4b656a5ce688f5c2406170782a0b /kernel/trace/trace_irqsoff.c | |
parent | aa1e0e3bcf95ce684d005bedb16e5d4559455685 (diff) |
ftrace: move pc counter in irqtrace
The assigning of the pc counter is in the wrong spot in the
check_critical_timing function. The pc variable is used in the
out jump.
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/trace/trace_irqsoff.c')
-rw-r--r-- | kernel/trace/trace_irqsoff.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace_irqsoff.c b/kernel/trace/trace_irqsoff.c index f925dbbff2a6..a7db7f040ae0 100644 --- a/kernel/trace/trace_irqsoff.c +++ b/kernel/trace/trace_irqsoff.c | |||
@@ -142,11 +142,11 @@ check_critical_timing(struct trace_array *tr, | |||
142 | 142 | ||
143 | local_save_flags(flags); | 143 | local_save_flags(flags); |
144 | 144 | ||
145 | pc = preempt_count(); | ||
146 | |||
145 | if (!report_latency(delta)) | 147 | if (!report_latency(delta)) |
146 | goto out; | 148 | goto out; |
147 | 149 | ||
148 | pc = preempt_count(); | ||
149 | |||
150 | spin_lock_irqsave(&max_trace_lock, flags); | 150 | spin_lock_irqsave(&max_trace_lock, flags); |
151 | 151 | ||
152 | /* check if we are still the max latency */ | 152 | /* check if we are still the max latency */ |