diff options
author | Carsten Emde <Carsten.Emde@osadl.org> | 2009-09-12 19:43:07 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2009-09-12 21:45:17 -0400 |
commit | b5130b1e7d3717d03ab1916b198bf0d49fa0a619 (patch) | |
tree | 67ee3282f6b6aa2e37864b938f67fd7e02b04394 /kernel/trace/trace_irqsoff.c | |
parent | 41dfba4367109b92d92ec6e059be6950497d932f (diff) |
tracing: do not update tracing_max_latency when tracer is stopped
The state of the function pair tracing_stop()/tracing_start() is
correctly considered when tracer data are updated. However, the global
and externally accessible variable tracing_max_latency is always updated
- even when tracing is stopped.
The update should only occur, if tracing was not stopped.
Signed-off-by: Carsten Emde <C.Emde@osadl.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_irqsoff.c')
-rw-r--r-- | kernel/trace/trace_irqsoff.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/kernel/trace/trace_irqsoff.c b/kernel/trace/trace_irqsoff.c index 06f8ea9e4b9d..3aa7eaa2114c 100644 --- a/kernel/trace/trace_irqsoff.c +++ b/kernel/trace/trace_irqsoff.c | |||
@@ -157,8 +157,10 @@ check_critical_timing(struct trace_array *tr, | |||
157 | 157 | ||
158 | data->critical_end = parent_ip; | 158 | data->critical_end = parent_ip; |
159 | 159 | ||
160 | tracing_max_latency = delta; | 160 | if (likely(!is_tracing_stopped())) { |
161 | update_max_tr_single(tr, current, cpu); | 161 | tracing_max_latency = delta; |
162 | update_max_tr_single(tr, current, cpu); | ||
163 | } | ||
162 | 164 | ||
163 | max_sequence++; | 165 | max_sequence++; |
164 | 166 | ||