lkml.org 
[lkml]   [2013]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] perf trace: Remove thread summary coloring
    Date
    Thread summary line coloring looks ugly.  It doesn't add much value so
    remove coloring completely.

    Signed-off-by: Pekka Enberg <penberg@kernel.org>
    ---
    tools/perf/builtin-trace.c | 13 ++-----------
    1 file changed, 2 insertions(+), 11 deletions(-)

    diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
    index 943087f..b03551e 100644
    --- a/tools/perf/builtin-trace.c
    +++ b/tools/perf/builtin-trace.c
    @@ -2150,7 +2150,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)
    @@ -2158,17 +2157,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);

    --
    1.8.3.1


    \
     
     \ /
      Last update: 2013-11-14 19:01    [W:4.362 / U:0.784 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site