Commit 15e65c69 authored by Pekka Enberg's avatar Pekka Enberg Committed by Arnaldo Carvalho de Melo
Browse files

perf trace: Remove thread summary coloring 



Thread summary line coloring looks ugly.  It doesn't add much value so
remove coloring completely.

Signed-off-by: default avatarPekka Enberg <penberg@kernel.org>
Acked-by: default avatarDavid Ahern <dsahern@gmail.com>
Link: http://lkml.kernel.org/r/1384447410-1771-1-git-send-email-penberg@kernel.org


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 12e55569

tools/perf/builtin-trace.c

+2 −11
Original line number Diff line number Diff line
@@ -2158,7 +2158,6 @@ static int trace__fprintf_one_thread(struct thread *thread, void *priv) 
	size_t printed = data->printed;

	struct trace *trace = data->trace;

	struct thread_trace *ttrace = thread->priv;

	const char *color;

	double ratio;



	if (ttrace == NULL)
@@ -2166,17 +2165,9 @@ static int trace__fprintf_one_thread(struct thread *thread, void *priv) 


	ratio = (double)ttrace->nr_events / trace->nr_events * 100.0;



	color = PERF_COLOR_NORMAL;

	if (ratio > 50.0)

		color = PERF_COLOR_RED;

	else if (ratio > 25.0)

		color = PERF_COLOR_GREEN;

	else if (ratio > 5.0)

		color = PERF_COLOR_YELLOW;



	printed += color_fprintf(fp, color, " %s (%d), ", thread__comm_str(thread), thread->tid);

	printed += fprintf(fp, " %s (%d), ", thread__comm_str(thread), thread->tid);

	printed += fprintf(fp, "%lu events, ", ttrace->nr_events);

	printed += color_fprintf(fp, color, "%.1f%%", ratio);

	printed += fprintf(fp, "%.1f%%", ratio);

	printed += fprintf(fp, ", %.3f msec\n", ttrace->runtime_ms);

	printed += thread__dump_stats(ttrace, trace, fp);

CRA Git | Maintained and supported by SUSTech CRA and CCSE