diff options
| author | Steven Rostedt <[email protected]> | 2016-03-23 10:16:28 -0400 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2016-03-23 15:08:12 -0300 |
| commit | 4f3c887688a2b0c01b241afcfedbb4e5e4a8e022 (patch) | |
| tree | 2d3b47c983532f3a0e94f0a2e7a45e1d82cb208d | |
| parent | 3938bad44ed2fea41328e4be2ae04a8e94540813 (diff) | |
tools lib traceevent: Remove redundant CPU output
Commit a6745330789f ("tools lib traceevent: Split pevent_print_event()
into specific functionality functions") broke apart the function
pevent_print_event() into three functions.
The first function prints the comm, pid and CPU, the second prints the
timestamp.
But that commit added the printing of the CPU in the timestamp function,
which now causes pevent_print_event() to duplicate the CPU output.
Remove the redundant printing of the record's CPU from the timestamp
function.
Signed-off-by: Steven Rostedt <[email protected]>
Cc: Namhyung Kim <[email protected]>
Fixes: a6745330789f ("tools lib traceevent: Split pevent_print_event() into specific functionality functions")
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
| -rw-r--r-- | tools/lib/traceevent/event-parse.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c index 190cc886ab91..a8b6357d1ffe 100644 --- a/tools/lib/traceevent/event-parse.c +++ b/tools/lib/traceevent/event-parse.c @@ -5427,10 +5427,8 @@ void pevent_print_event_time(struct pevent *pevent, struct trace_seq *s, } if (pevent->latency_format) { - trace_seq_printf(s, " %3d", record->cpu); pevent_data_lat_fmt(pevent, s, record); - } else - trace_seq_printf(s, " [%03d]", record->cpu); + } if (use_usec_format) { if (pevent->flags & PEVENT_NSEC_OUTPUT) { |