perf report: Honor column width setting

Set column width and do not change it if user gives -w/--column-widths
option.  It'll truncate longer symbols than the width if exists.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1406785662-5534-5-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

4 files changed, 77 insertions, 39 deletions

diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c
index c1d8d3925fe1..e07d4e848d5c 100644
--- a/tools/perf/ui/browsers/hists.c
+++ b/tools/perf/ui/browsers/hists.c
@@ -678,12 +678,12 @@ static u64 __hpp_get_##_field(struct hist_entry *he)			\
 }                                                                       \
                                                                         \
 static int                                                              \
-hist_browser__hpp_color_##_type(struct perf_hpp_fmt *fmt __maybe_unused,\
+hist_browser__hpp_color_##_type(struct perf_hpp_fmt *fmt,               \
                                 struct perf_hpp *hpp,                   \
                                 struct hist_entry *he)                  \
 {                                                                       \
-        return __hpp__fmt(hpp, he, __hpp_get_##_field, " %*.2f%%", 6,   \
-                          __hpp__slsmg_color_printf, true);             \
+        return hpp__fmt(fmt, hpp, he, __hpp_get_##_field, " %*.2f%%",   \
+                        __hpp__slsmg_color_printf, true);               \
 }
 
 #define __HPP_COLOR_ACC_PERCENT_FN(_type, _field)                       \
@@ -693,19 +693,20 @@ static u64 __hpp_get_acc_##_field(struct hist_entry *he)		\
 }                                                                       \
                                                                         \
 static int                                                              \
-hist_browser__hpp_color_##_type(struct perf_hpp_fmt *fmt __maybe_unused,\
+hist_browser__hpp_color_##_type(struct perf_hpp_fmt *fmt,               \
                                 struct perf_hpp *hpp,                   \
                                 struct hist_entry *he)                  \
 {                                                                       \
         if (!symbol_conf.cumulate_callchain) {                          \
+                int len = fmt->user_len ?: fmt->len;                    \
                 int ret = scnprintf(hpp->buf, hpp->size,                \
-                                    "%*s", 8, "N/A");                   \
+                                    "%*s", len, "N/A");                 \
                 slsmg_printf("%s", hpp->buf);                           \
                                                                         \
                 return ret;                                             \
         }                                                               \
-        return __hpp__fmt(hpp, he, __hpp_get_acc_##_field, " %*.2f%%",  \
-                          6, __hpp__slsmg_color_printf, true);  \
+        return hpp__fmt(fmt, hpp, he, __hpp_get_acc_##_field,           \
+                        " %*.2f%%", __hpp__slsmg_color_printf, true);   \
 }
 
 __HPP_COLOR_PERCENT_FN(overhead, period)
@@ -1549,6 +1550,9 @@ static int perf_evsel__hists_browse(struct perf_evsel *evsel, int nr_events,
 
         memset(options, 0, sizeof(options));
 
+        if (symbol_conf.col_width_list_str)
+                perf_hpp__set_user_width(symbol_conf.col_width_list_str);
+
         while (1) {
                 const struct thread *thread = NULL;
                 const struct dso *dso = NULL;
diff --git a/tools/perf/ui/gtk/hists.c b/tools/perf/ui/gtk/hists.c
index 91f6cd7d2312..897b2e140428 100644
--- a/tools/perf/ui/gtk/hists.c
+++ b/tools/perf/ui/gtk/hists.c
@@ -41,12 +41,12 @@ static u64 he_get_##_field(struct hist_entry *he)				\
         return he->stat._field;                                                 \
 }                                                                               \
                                                                                 \
-static int perf_gtk__hpp_color_##_type(struct perf_hpp_fmt *fmt __maybe_unused, \
+static int perf_gtk__hpp_color_##_type(struct perf_hpp_fmt *fmt,                \
                                        struct perf_hpp *hpp,                    \
                                        struct hist_entry *he)                   \
 {                                                                               \
-        return __hpp__fmt(hpp, he, he_get_##_field, " %*.2f%%", 6,              \
-                          __percent_color_snprintf, true);                      \
+        return hpp__fmt(fmt, hpp, he, he_get_##_field, " %*.2f%%",              \
+                        __percent_color_snprintf, true);                        \
 }
 
 #define __HPP_COLOR_ACC_PERCENT_FN(_type, _field)                               \
@@ -59,8 +59,8 @@ static int perf_gtk__hpp_color_##_type(struct perf_hpp_fmt *fmt __maybe_unused,
                                        struct perf_hpp *hpp,                    \
                                        struct hist_entry *he)                   \
 {                                                                               \
-        return __hpp__fmt_acc(hpp, he, he_get_acc_##_field, " %*.2f%%", 6,      \
-                              __percent_color_snprintf, true);                  \
+        return hpp__fmt_acc(fmt, hpp, he, he_get_acc_##_field, " %*.2f%%",      \
+                            __percent_color_snprintf, true);                    \
 }
 
 __HPP_COLOR_PERCENT_FN(overhead, period)
diff --git a/tools/perf/ui/hist.c b/tools/perf/ui/hist.c
index e28ca972d046..b2d60a95f01d 100644
--- a/tools/perf/ui/hist.c
+++ b/tools/perf/ui/hist.c
@@ -15,9 +15,9 @@
         __ret;                                                  \
 })
 
-int __hpp__fmt(struct perf_hpp *hpp, struct hist_entry *he,
-               hpp_field_fn get_field, const char *fmt, int len,
-               hpp_snprint_fn print_fn, bool fmt_percent)
+static int __hpp__fmt(struct perf_hpp *hpp, struct hist_entry *he,
+                      hpp_field_fn get_field, const char *fmt, int len,
+                      hpp_snprint_fn print_fn, bool fmt_percent)
 {
         int ret;
         struct hists *hists = he->hists;
@@ -104,16 +104,35 @@ int __hpp__fmt(struct perf_hpp *hpp, struct hist_entry *he,
         return ret;
 }
 
-int __hpp__fmt_acc(struct perf_hpp *hpp, struct hist_entry *he,
-                   hpp_field_fn get_field, const char *fmt, int len,
-                   hpp_snprint_fn print_fn, bool fmt_percent)
+int hpp__fmt(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
+             struct hist_entry *he, hpp_field_fn get_field,
+             const char *fmtstr, hpp_snprint_fn print_fn, bool fmt_percent)
+{
+        int len = fmt->user_len ?: fmt->len;
+
+        if (symbol_conf.field_sep) {
+                return __hpp__fmt(hpp, he, get_field, fmtstr, 1,
+                                  print_fn, fmt_percent);
+        }
+
+        if (fmt_percent)
+                len -= 2; /* 2 for a space and a % sign */
+        else
+                len -= 1;
+
+        return  __hpp__fmt(hpp, he, get_field, fmtstr, len, print_fn, fmt_percent);
+}
+
+int hpp__fmt_acc(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
+                 struct hist_entry *he, hpp_field_fn get_field,
+                 const char *fmtstr, hpp_snprint_fn print_fn, bool fmt_percent)
 {
         if (!symbol_conf.cumulate_callchain) {
-                return snprintf(hpp->buf, hpp->size, "%*s",
-                                fmt_percent ? len + 2 : len + 1, "N/A");
+                int len = fmt->user_len ?: fmt->len;
+                return snprintf(hpp->buf, hpp->size, " %*s", len - 1, "N/A");
         }
 
-        return __hpp__fmt(hpp, he, get_field, fmt, len, print_fn, fmt_percent);
+        return hpp__fmt(fmt, hpp, he, get_field, fmtstr, print_fn, fmt_percent);
 }
 
 static int field_cmp(u64 field_a, u64 field_b)
@@ -195,10 +214,10 @@ static int hpp__width_##_type(struct perf_hpp_fmt *fmt,			\
                               struct perf_hpp *hpp __maybe_unused,      \
                               struct perf_evsel *evsel)                 \
 {                                                                       \
-        int len = fmt->len;                                             \
+        int len = fmt->user_len ?: fmt->len;                            \
                                                                         \
         if (symbol_conf.event_group)                                    \
-                len = max(len, evsel->nr_members * len);                \
+                len = max(len, evsel->nr_members * fmt->len);           \
                                                                         \
         if (len < (int)strlen(_str))                                    \
                 len = strlen(_str);                                     \
@@ -253,18 +272,16 @@ static u64 he_get_##_field(struct hist_entry *he)				\
 static int hpp__color_##_type(struct perf_hpp_fmt *fmt,                         \
                               struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                               \
-        int len = fmt->len - 2; /* 2 for a space and a % sign */                \
-        return __hpp__fmt(hpp, he, he_get_##_field, " %*.2f%%", len,            \
-                          hpp_color_scnprintf, true);                           \
+        return hpp__fmt(fmt, hpp, he, he_get_##_field, " %*.2f%%",              \
+                        hpp_color_scnprintf, true);                             \
 }
 
 #define __HPP_ENTRY_PERCENT_FN(_type, _field)                                   \
 static int hpp__entry_##_type(struct perf_hpp_fmt *fmt,                         \
                               struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                               \
-        int len = symbol_conf.field_sep ? 1 : fmt->len - 2;                     \
-        return __hpp__fmt(hpp, he, he_get_##_field, " %*.2f%%", len,            \
-                          hpp_entry_scnprintf, true);                           \
+        return hpp__fmt(fmt, hpp, he, he_get_##_field, " %*.2f%%",              \
+                        hpp_entry_scnprintf, true);                             \
 }
 
 #define __HPP_SORT_FN(_type, _field)                                            \
@@ -282,18 +299,16 @@ static u64 he_get_acc_##_field(struct hist_entry *he)				\
 static int hpp__color_##_type(struct perf_hpp_fmt *fmt,                         \
                               struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                               \
-        int len = fmt->len - 2; /* 2 for a space and a % sign */                \
-        return __hpp__fmt_acc(hpp, he, he_get_acc_##_field, " %*.2f%%", len,    \
-                              hpp_color_scnprintf, true);                       \
+        return hpp__fmt_acc(fmt, hpp, he, he_get_acc_##_field, " %*.2f%%",      \
+                            hpp_color_scnprintf, true);                         \
 }
 
 #define __HPP_ENTRY_ACC_PERCENT_FN(_type, _field)                               \
 static int hpp__entry_##_type(struct perf_hpp_fmt *fmt,                         \
                               struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                               \
-        int len = symbol_conf.field_sep ? 1 : fmt->len - 2;                     \
-        return __hpp__fmt_acc(hpp, he, he_get_##_field, " %*.2f%%", len,        \
-                              hpp_entry_scnprintf, true);                       \
+        return hpp__fmt_acc(fmt, hpp, he, he_get_##_field, " %*.2f%%",          \
+                            hpp_entry_scnprintf, true);                         \
 }
 
 #define __HPP_SORT_ACC_FN(_type, _field)                                        \
@@ -311,9 +326,8 @@ static u64 he_get_raw_##_field(struct hist_entry *he)				\
 static int hpp__entry_##_type(struct perf_hpp_fmt *fmt,                         \
                               struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                               \
-        int len = symbol_conf.field_sep ? 1 : fmt->len - 1;                     \
-        return __hpp__fmt(hpp, he, he_get_raw_##_field, " %*"PRIu64, len,       \
-                          hpp_entry_scnprintf, false);                          \
+        return hpp__fmt(fmt, hpp, he, he_get_raw_##_field, " %*"PRIu64,         \
+                        hpp_entry_scnprintf, false);                            \
 }
 
 #define __HPP_SORT_RAW_FN(_type, _field)                                        \
@@ -666,3 +680,21 @@ void perf_hpp__reset_width(struct perf_hpp_fmt *fmt, struct hists *hists)
                 break;
         }
 }
+
+void perf_hpp__set_user_width(const char *width_list_str)
+{
+        struct perf_hpp_fmt *fmt;
+        const char *ptr = width_list_str;
+
+        perf_hpp__for_each_format(fmt) {
+                char *p;
+
+                int len = strtol(ptr, &p, 10);
+                fmt->user_len = len;
+
+                if (*p == ',')
+                        ptr = p + 1;
+                else
+                        break;
+        }
+}
diff --git a/tools/perf/ui/stdio/hist.c b/tools/perf/ui/stdio/hist.c
index 40af0acb4fe9..15b451acbde6 100644
--- a/tools/perf/ui/stdio/hist.c
+++ b/tools/perf/ui/stdio/hist.c
@@ -395,10 +395,12 @@ size_t hists__fprintf(struct hists *hists, bool show_header, int max_rows,
 
         init_rem_hits();
 
-
         perf_hpp__for_each_format(fmt)
                 perf_hpp__reset_width(fmt, hists);
 
+        if (symbol_conf.col_width_list_str)
+                perf_hpp__set_user_width(symbol_conf.col_width_list_str);
+
         if (!show_header)
                 goto print_entries;