summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJiri Olsa <jolsa@redhat.com>2012-12-01 21:57:04 +0100
committerArnaldo Carvalho de Melo <acme@redhat.com>2013-07-12 13:54:09 -0300
commitc818b49820aea96d6a1b43815bae0ee38b09ca0d (patch)
tree3d7a52966a518d384d8f560f30ba8143b05a8622
parent345dc0b45ecc37a239723f2b6392cab04d8b0eff (diff)
downloadop-kernel-dev-c818b49820aea96d6a1b43815bae0ee38b09ca0d.zip
op-kernel-dev-c818b49820aea96d6a1b43815bae0ee38b09ca0d.tar.gz
perf diff: Move columns into struct data__file
Another step towards multiple data files support. Having columns definition within struct data__file force each data file having its own columns. Signed-off-by: Jiri Olsa <jolsa@redhat.com> Reviewed-by: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Paul Mackerras <paulus@samba.org> Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Namhyung Kim <namhyung@kernel.org> Link: http://lkml.kernel.org/n/tip-lnfqj7k7fqw8bz07pupi5464@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r--tools/perf/builtin-diff.c97
1 files changed, 55 insertions, 42 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c
index 8734f1c..7787ee2 100644
--- a/tools/perf/builtin-diff.c
+++ b/tools/perf/builtin-diff.c
@@ -44,6 +44,7 @@ struct data__file {
struct perf_session *session;
const char *file;
int idx;
+ struct diff_hpp_fmt fmt[PERF_HPP_DIFF__MAX_INDEX];
};
static struct data__file *data__files;
@@ -584,6 +585,17 @@ static void data_process(void)
}
}
+static void data__free(struct data__file *d)
+{
+ int col;
+
+ for (col = 0; col < PERF_HPP_DIFF__MAX_INDEX; col++) {
+ struct diff_hpp_fmt *fmt = &d->fmt[col];
+
+ free(fmt->header);
+ }
+}
+
static int __cmd_diff(void)
{
struct data__file *d;
@@ -613,6 +625,8 @@ static int __cmd_diff(void)
data__for_each_file(i, d) {
if (d->session)
perf_session__delete(d->session);
+
+ data__free(d);
}
free(data__files);
@@ -818,32 +832,6 @@ static int hpp__width(struct perf_hpp_fmt *fmt,
return dfmt->header_width;
}
-#define hpp__color_global hpp__entry_global
-
-#define FMT(_i, _entry, _color) \
- [_i] = { \
- .fmt = { \
- .header = hpp__header, \
- .width = hpp__width, \
- .entry = hpp__entry_ ## _entry, \
- .color = hpp__color_ ## _color, \
- }, \
- .idx = _i, \
- }
-
-#define FMT_GLOBAL(_i) FMT(_i, global, global)
-#define FMT_BASELINE(_i) FMT(_i, global, baseline)
-
-static struct diff_hpp_fmt diff_fmt[] = {
- FMT_BASELINE(PERF_HPP_DIFF__BASELINE),
- FMT_GLOBAL(PERF_HPP_DIFF__PERIOD),
- FMT_GLOBAL(PERF_HPP_DIFF__PERIOD_BASELINE),
- FMT_GLOBAL(PERF_HPP_DIFF__DELTA),
- FMT_GLOBAL(PERF_HPP_DIFF__RATIO),
- FMT_GLOBAL(PERF_HPP_DIFF__WEIGHTED_DIFF),
- FMT_GLOBAL(PERF_HPP_DIFF__FORMULA),
-};
-
static void init_header(struct diff_hpp_fmt *dfmt)
{
#define MAX_HEADER_NAME 100
@@ -873,31 +861,56 @@ static void init_header(struct diff_hpp_fmt *dfmt)
#undef NAME
}
-static void column_enable(unsigned col)
+static void data__hpp_register(struct data__file *d, int idx)
{
- struct diff_hpp_fmt *dfmt;
+ struct diff_hpp_fmt *dfmt = &d->fmt[idx];
+ struct perf_hpp_fmt *fmt = &dfmt->fmt;
+
+ dfmt->idx = idx;
+
+ fmt->header = hpp__header;
+ fmt->width = hpp__width;
+ fmt->entry = hpp__entry_global;
+
+ /* TODO more colors */
+ if (idx == PERF_HPP_DIFF__BASELINE)
+ fmt->color = hpp__color_baseline;
- BUG_ON(col >= PERF_HPP_DIFF__MAX_INDEX);
- dfmt = &diff_fmt[col];
init_header(dfmt);
- perf_hpp__column_register(&dfmt->fmt);
+ perf_hpp__column_register(fmt);
}
static void ui_init(void)
{
- /*
- * Display baseline/delta/ratio/
- * formula/periods columns.
- */
- column_enable(PERF_HPP_DIFF__BASELINE);
- column_enable(compute_2_hpp[compute]);
+ struct data__file *d;
+ int i;
+
+ data__for_each_file(i, d) {
+
+ /*
+ * Baseline or compute realted columns:
+ *
+ * PERF_HPP_DIFF__BASELINE
+ * PERF_HPP_DIFF__DELTA
+ * PERF_HPP_DIFF__RATIO
+ * PERF_HPP_DIFF__WEIGHTED_DIFF
+ */
+ data__hpp_register(d, i ? compute_2_hpp[compute] :
+ PERF_HPP_DIFF__BASELINE);
- if (show_formula)
- column_enable(PERF_HPP_DIFF__FORMULA);
+ /*
+ * And the rest:
+ *
+ * PERF_HPP_DIFF__FORMULA
+ * PERF_HPP_DIFF__PERIOD
+ * PERF_HPP_DIFF__PERIOD_BASELINE
+ */
+ if (show_formula && i)
+ data__hpp_register(d, PERF_HPP_DIFF__FORMULA);
- if (show_period) {
- column_enable(PERF_HPP_DIFF__PERIOD);
- column_enable(PERF_HPP_DIFF__PERIOD_BASELINE);
+ if (show_period)
+ data__hpp_register(d, i ? PERF_HPP_DIFF__PERIOD :
+ PERF_HPP_DIFF__PERIOD_BASELINE);
}
}
OpenPOWER on IntegriCloud