* [PATCH][GIT PULL][v3.4] tracing: Fix stacktrace of latency tracers (irqsoff and friends)
@ 2012-04-19 21:06 Steven Rostedt
2012-04-25 10:35 ` Ingo Molnar
0 siblings, 1 reply; 2+ messages in thread
From: Steven Rostedt @ 2012-04-19 21:06 UTC (permalink / raw)
To: LKML
Cc: Ingo Molnar, Thomas Gleixner, Frederic Weisbecker, Andrew Morton, stable
[-- Attachment #1: Type: text/plain, Size: 2860 bytes --]
Ingo,
Found another stacktrace bug. This is on top of the last pull request I
made.
-- Steve
Please pull the latest tip/perf/urgent-2 tree, which can be found at:
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
tip/perf/urgent-2
Head SHA1: db4c75cbebd7e5910cd3bcb6790272fcc3042857
Steven Rostedt (1):
tracing: Fix stacktrace of latency tracers (irqsoff and friends)
----
kernel/trace/trace_output.c | 5 +++++
1 file changed, 5 insertions(+)
---------------------------
commit db4c75cbebd7e5910cd3bcb6790272fcc3042857
Author: Steven Rostedt <srostedt@redhat.com>
Date: Thu Apr 19 10:31:47 2012 -0400
tracing: Fix stacktrace of latency tracers (irqsoff and friends)
While debugging a latency with someone on IRC (mirage335) on #linux-rt (OFTC),
we discovered that the stacktrace output of the latency tracers
(preemptirqsoff) was empty.
This bug was caused by the creation of the dynamic length stack trace
again (like commit 12b5da3 "tracing: Fix ent_size in trace output" was).
This bug is caused by the latency tracers requiring the next event
to determine the time between the current event and the next. But by
grabbing the next event, the iter->ent_size is set to the next event
instead of the current one. As the stacktrace event is the last event,
this makes the ent_size zero and causes nothing to be printed for
the stack trace. The dynamic stacktrace uses the ent_size to determine
how much of the stack can be printed. The ent_size of zero means
no stack.
The simple fix is to save the iter->ent_size before finding the next event.
Note, mirage335 asked to remain anonymous from LKML and git, so I will
not add the Reported-by and Tested-by tags, even though he did report
the issue and tested the fix.
Cc: stable@vger.kernel.org # 3.1+
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
diff --git a/kernel/trace/trace_output.c b/kernel/trace/trace_output.c
index 859fae6..df611a0 100644
--- a/kernel/trace/trace_output.c
+++ b/kernel/trace/trace_output.c
@@ -652,6 +652,8 @@ int trace_print_lat_context(struct trace_iterator *iter)
{
u64 next_ts;
int ret;
+ /* trace_find_next_entry will reset ent_size */
+ int ent_size = iter->ent_size;
struct trace_seq *s = &iter->seq;
struct trace_entry *entry = iter->ent,
*next_entry = trace_find_next_entry(iter, NULL,
@@ -660,6 +662,9 @@ int trace_print_lat_context(struct trace_iterator *iter)
unsigned long abs_usecs = ns2usecs(iter->ts - iter->tr->time_start);
unsigned long rel_usecs;
+ /* Restore the original ent_size */
+ iter->ent_size = ent_size;
+
if (!next_entry)
next_ts = iter->ts;
rel_usecs = ns2usecs(next_ts - iter->ts);
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH][GIT PULL][v3.4] tracing: Fix stacktrace of latency tracers (irqsoff and friends)
2012-04-19 21:06 [PATCH][GIT PULL][v3.4] tracing: Fix stacktrace of latency tracers (irqsoff and friends) Steven Rostedt
@ 2012-04-25 10:35 ` Ingo Molnar
0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2012-04-25 10:35 UTC (permalink / raw)
To: Steven Rostedt
Cc: LKML, Ingo Molnar, Thomas Gleixner, Frederic Weisbecker,
Andrew Morton, stable
* Steven Rostedt <rostedt@goodmis.org> wrote:
>
> Ingo,
>
> Found another stacktrace bug. This is on top of the last pull request I
> made.
>
> -- Steve
>
>
> Please pull the latest tip/perf/urgent-2 tree, which can be found at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
> tip/perf/urgent-2
>
> Head SHA1: db4c75cbebd7e5910cd3bcb6790272fcc3042857
>
>
> Steven Rostedt (1):
> tracing: Fix stacktrace of latency tracers (irqsoff and friends)
>
> ----
> kernel/trace/trace_output.c | 5 +++++
> 1 file changed, 5 insertions(+)
Pulled, thanks Steve!
Ingo
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-04-25 10:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-19 21:06 [PATCH][GIT PULL][v3.4] tracing: Fix stacktrace of latency tracers (irqsoff and friends) Steven Rostedt
2012-04-25 10:35 ` Ingo Molnar
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.