diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-05-15 05:19:09 +0200 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-05-15 05:19:09 +0200 |
| commit | 29a679754b1a2581ee456eada6c2de7ce95068bb (patch) | |
| tree | e647c89f3a293f4436ed285acc46180e6fe9a292 /kernel/trace/trace_output.c | |
| parent | ring-buffer: move code around to remove some branches (diff) | |
| download | linux-29a679754b1a2581ee456eada6c2de7ce95068bb.tar.xz linux-29a679754b1a2581ee456eada6c2de7ce95068bb.zip | |
x86/stacktrace: return 0 instead of -1 for stack ops
If we return -1 in the ops->stack for the stacktrace saving, we end up
breaking out of the loop if the stack we are tracing is in the exception
stack. This causes traces like:
<idle>-0 [002] 34263.745825: raise_softirq_irqoff <-__blk_complete_request
<idle>-0 [002] 34263.745826:
<= 0
<= 0
<= 0
<= 0
<= 0
<= 0
<= 0
By returning "0" instead, the irq stack is saved as well, and we see:
<idle>-0 [003] 883.280992: raise_softirq_irqoff <-__hrtimer_star
t_range_ns
<idle>-0 [003] 883.280992:
<= hrtimer_start_range_ns
<= tick_nohz_restart_sched_tick
<= cpu_idle
<= start_secondary
<=
<= 0
<= 0
[ Impact: record stacks from interrupts ]
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_output.c')
0 files changed, 0 insertions, 0 deletions