perf sched timehist: Honour 'comm_width' when aligning the headers
authorNamhyung Kim <namhyung@kernel.org>
Thu, 22 Dec 2016 06:03:48 +0000 (15:03 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 22 Dec 2016 19:35:45 +0000 (16:35 -0300)
Current default value is 20, but that may change in the future, so make
places where we have 20 hardcoded use 'comm_width'.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20161222060350.17655-1-namhyung@kernel.org
[ Split from a larger patch ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-sched.c

index c1c07bfe132ca978d6dc67ce84cac883d9031810..020f9c9cef7b9bb74c50cdda2a5e870c261885b7 100644 (file)
@@ -1817,7 +1817,7 @@ static void timehist_header(struct perf_sched *sched)
                printf(" ");
        }
 
-       printf(" %-20s  %9s  %9s  %9s",
+       printf(" %-*s  %9s  %9s  %9s", comm_width,
                "task name", "wait time", "sch delay", "run time");
 
        printf("\n");
@@ -1830,7 +1830,8 @@ static void timehist_header(struct perf_sched *sched)
        if (sched->show_cpu_visual)
                printf(" %*s ", ncpus, "");
 
-       printf(" %-20s  %9s  %9s  %9s\n", "[tid/pid]", "(msec)", "(msec)", "(msec)");
+       printf(" %-*s  %9s  %9s  %9s\n", comm_width,
+              "[tid/pid]", "(msec)", "(msec)", "(msec)");
 
        /*
         * separator
@@ -1840,7 +1841,7 @@ static void timehist_header(struct perf_sched *sched)
        if (sched->show_cpu_visual)
                printf(" %.*s ", ncpus, graph_dotted_line);
 
-       printf(" %.20s  %.9s  %.9s  %.9s",
+       printf(" %.*s  %.9s  %.9s  %.9s", comm_width,
                graph_dotted_line, graph_dotted_line, graph_dotted_line,
                graph_dotted_line);