Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

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 <rostedt@goodmis.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Fixes: a6745330789f ("tools lib traceevent: Split pevent_print_event() into specific functionality functions")
Link: http://lkml.kernel.org/r/20160323101628.459375d2@gandalf.local.home
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

authored by

Steven Rostedt and committed by
Arnaldo Carvalho de Melo
4f3c8876 3938bad4

+1 -3
+1 -3
tools/lib/traceevent/event-parse.c
··· 5427 5427 } 5428 5428 5429 5429 if (pevent->latency_format) { 5430 - trace_seq_printf(s, " %3d", record->cpu); 5431 5430 pevent_data_lat_fmt(pevent, s, record); 5432 - } else 5433 - trace_seq_printf(s, " [%03d]", record->cpu); 5431 + } 5434 5432 5435 5433 if (use_usec_format) { 5436 5434 if (pevent->flags & PEVENT_NSEC_OUTPUT) {