diff options
| author | Steven Rostedt (Google) <rostedt@goodmis.org> | 2024-01-08 19:17:59 -0500 |
|---|---|---|
| committer | Steven Rostedt (Google) <rostedt@goodmis.org> | 2024-01-10 13:49:49 -0500 |
| commit | 1e713bfc3dd56846cc84f6cfeb4ce88e843569dd (patch) | |
| tree | a6ae9eae0ab24c514e984d6073e81d7b2ddeb6e4 | |
| parent | b86599e459618f858d5d596ac3854a8c529fad84 (diff) | |
| download | trace-cmd-1e713bfc3dd56846cc84f6cfeb4ce88e843569dd.tar.gz | |
trace-cmd report: Add interrupt info in output
In the Linux kernel version 3.3 the default output showed the irq information
that is typically only for the latency format. That is, it shows:
<idle>-0 [000] d..2 49.309305: cpuidle_get_driver <-cpuidle_idle_call
<idle>-0 [000] d..2 49.309307: mwait_idle <-cpu_idle
<idle>-0 [000] d..2 49.309309: need_resched <-mwait_idle
Instead of:
<idle>-0 [000] 49.309305: cpuidle_get_driver <-cpuidle_idle_call
<idle>-0 [000] 49.309307: mwait_idle <-cpu_idle
<idle>-0 [000] 49.309309: need_resched <-mwait_idle
After 12 years, it's about time that the default trace-cmd report output did
the same!
This does change the output format and possibly can break anything that
tries to parse it.
Link: https://lore.kernel.org/linux-trace-devel/20240108191759.5fc4ecda@gandalf.local.home
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
| -rw-r--r-- | tracecmd/trace-read.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/tracecmd/trace-read.c b/tracecmd/trace-read.c index f645c053..1dea54c4 100644 --- a/tracecmd/trace-read.c +++ b/tracecmd/trace-read.c @@ -870,14 +870,13 @@ void trace_show_data(struct tracecmd_input *handle, struct tep_record *record) TEP_PRINT_PID, TEP_PRINT_CPU); - if (latency_format) { - if (raw_format) - trace_seq_printf(&s, "-0x%x", - tep_data_flags(pevent, record)); - else - tep_print_event(pevent, &s, record, "%s", - TEP_PRINT_LATENCY); - } + if (raw_format) + trace_seq_printf(&s, "-0x%x", + tep_data_flags(pevent, record)); + else + tep_print_event(pevent, &s, record, + latency_format ? "%s" : " %s", + TEP_PRINT_LATENCY); tep_print_event(pevent, &s, record, tfmt, TEP_PRINT_TIME); |