perf report: Add raw displaying of per-thread counters

If --pretty=raw is given to perf report -T, it now displays one
line per-thread per-counter with the raw event id added.

We get:
 #   PID    TID              Name  Raw    Count
   18608  18609      cache-misses  28e   416744
   18608  18609  cache-references  28f  6456792
   18608  18608      cache-misses  28e   448219
   18608  18608  cache-references  28f  7270244
 instead of:

#   PID    TID  cache-misses  cache-references
   18608  18609        416744           6456792
   18608  18608        448219           7270244

Signed-off-by: Brice Goglin <Brice.Goglin@inria.fr>
Acked-by: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <4A802008.5050409@inria.fr>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 4163918..2357c66 100644 (file)
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -57,6 +57,9 @@ static int            show_nr_samples;
 static int             show_threads;
 static struct perf_read_values show_threads_values;
 
+static char            default_pretty_printing_style[] = "normal";
+static char            *pretty_printing_style = default_pretty_printing_style;
+
 static unsigned long   page_size;
 static unsigned long   mmap_window = 32;
 
@@ -1401,6 +1404,9 @@ static size_t output__fprintf(FILE *fp, u64 total_samples)
        size_t ret = 0;
        unsigned int width;
        char *col_width = col_width_list_str;
+       int raw_printing_style;
+
+       raw_printing_style = !strcmp(pretty_printing_style, "raw");
 
        init_rem_hits();
 
@@ -1478,7 +1484,8 @@ print_entries:
        free(rem_sq_bracket);
 
        if (show_threads)
-               perf_read_values_display(fp, &show_threads_values);
+               perf_read_values_display(fp, &show_threads_values,
+                                        raw_printing_style);
 
        return ret;
 }
@@ -2091,6 +2098,8 @@ static const struct option options[] = {
                    "Show a column with the number of samples"),
        OPT_BOOLEAN('T', "threads", &show_threads,
                    "Show per-thread event counters"),
+       OPT_STRING(0, "pretty", &pretty_printing_style, "key",
+                  "pretty printing style key: normal raw"),
        OPT_STRING('s', "sort", &sort_order, "key[,key2...]",
                   "sort by key(s): pid, comm, dso, symbol, parent"),
        OPT_BOOLEAN('P', "full-paths", &full_paths,

diff --git a/tools/perf/util/values.c b/tools/perf/util/values.c
index 8551c0b..614cfaf 100644 (file)
--- a/tools/perf/util/values.c
+++ b/tools/perf/util/values.c
@@ -126,7 +126,8 @@ void perf_read_values_add_value(struct perf_read_values *values,
        values->value[tindex][cindex] = value;
 }
 
-void perf_read_values_display(FILE *fp, struct perf_read_values *values)
+static void perf_read_values__display_pretty(FILE *fp,
+                                            struct perf_read_values *values)
 {
        int i, j;
        int pidwidth, tidwidth;
@@ -169,3 +170,62 @@ void perf_read_values_display(FILE *fp, struct perf_read_values *values)
                fprintf(fp, "\n");
        }
 }
+
+static void perf_read_values__display_raw(FILE *fp,
+                                         struct perf_read_values *values)
+{
+       int width, pidwidth, tidwidth, namewidth, rawwidth, countwidth;
+       int i, j;
+
+       tidwidth = 3; /* TID */
+       pidwidth = 3; /* PID */
+       namewidth = 4; /* "Name" */
+       rawwidth = 3; /* "Raw" */
+       countwidth = 5; /* "Count" */
+
+       for (i = 0; i < values->threads; i++) {
+               width = snprintf(NULL, 0, "%d", values->pid[i]);
+               if (width > pidwidth)
+                       pidwidth = width;
+               width = snprintf(NULL, 0, "%d", values->tid[i]);
+               if (width > tidwidth)
+                       tidwidth = width;
+       }
+       for (j = 0; j < values->counters; j++) {
+               width = strlen(values->countername[j]);
+               if (width > namewidth)
+                       namewidth = width;
+               width = snprintf(NULL, 0, "%llx", values->counterrawid[j]);
+               if (width > rawwidth)
+                       rawwidth = width;
+       }
+       for (i = 0; i < values->threads; i++) {
+               for (j = 0; j < values->counters; j++) {
+                       width = snprintf(NULL, 0, "%Lu", values->value[i][j]);
+                       if (width > countwidth)
+                               countwidth = width;
+               }
+       }
+
+       fprintf(fp, "# %*s  %*s  %*s  %*s  %*s\n",
+               pidwidth, "PID", tidwidth, "TID",
+               namewidth, "Name", rawwidth, "Raw",
+               countwidth, "Count");
+       for (i = 0; i < values->threads; i++)
+               for (j = 0; j < values->counters; j++)
+                       fprintf(fp, "  %*d  %*d  %*s  %*llx  %*Lu\n",
+                               pidwidth, values->pid[i],
+                               tidwidth, values->tid[i],
+                               namewidth, values->countername[j],
+                               rawwidth, values->counterrawid[j],
+                               countwidth, values->value[i][j]);
+}
+
+void perf_read_values_display(FILE *fp, struct perf_read_values *values,
+                             int raw)
+{
+       if (raw)
+               perf_read_values__display_raw(fp, values);
+       else
+               perf_read_values__display_pretty(fp, values);
+}

diff --git a/tools/perf/util/values.h b/tools/perf/util/values.h
index e41be5e..f8960fd 100644 (file)
--- a/tools/perf/util/values.h
+++ b/tools/perf/util/values.h
@@ -21,6 +21,7 @@ void perf_read_values_add_value(struct perf_read_values *values,
                                u32 pid, u32 tid,
                                u64 rawid, char *name, u64 value);
 
-void perf_read_values_display(FILE *fp, struct perf_read_values *values);
+void perf_read_values_display(FILE *fp, struct perf_read_values *values,
+                             int raw);
 
 #endif /* _PERF_VALUES_H */