diff options
author | Namhyung Kim <namhyung@kernel.org> | 2017-02-10 11:18:56 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 12:29:38 -0500 |
commit | be57b3fd218ad4a19725ac4bd53e67b2ede42a9d (patch) | |
tree | 804aeae4cc480ff051cc0ff2b6e815080ecc9fa9 /tools/perf/builtin-diff.c | |
parent | 4b35994abe459f08f58b4b3855abf4ba80308680 (diff) |
perf diff: Change default setting to "delta-abs"
The "delta-abs" compute method will show most changed entries on top.
So users can easily see how much effect between the data. Note that it
also changes the default of -o option to 1 in order to apply the compute
method. To see original-style (sorted by baseline) use -o 0 option.
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20170210161856.18422-1-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-diff.c')
-rw-r--r-- | tools/perf/builtin-diff.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index e68cc76bdc5a..70a289347591 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c | |||
@@ -66,7 +66,7 @@ static bool force; | |||
66 | static bool show_period; | 66 | static bool show_period; |
67 | static bool show_formula; | 67 | static bool show_formula; |
68 | static bool show_baseline_only; | 68 | static bool show_baseline_only; |
69 | static unsigned int sort_compute; | 69 | static unsigned int sort_compute = 1; |
70 | 70 | ||
71 | static s64 compute_wdiff_w1; | 71 | static s64 compute_wdiff_w1; |
72 | static s64 compute_wdiff_w2; | 72 | static s64 compute_wdiff_w2; |
@@ -86,7 +86,7 @@ const char *compute_names[COMPUTE_MAX] = { | |||
86 | [COMPUTE_WEIGHTED_DIFF] = "wdiff", | 86 | [COMPUTE_WEIGHTED_DIFF] = "wdiff", |
87 | }; | 87 | }; |
88 | 88 | ||
89 | static int compute = COMPUTE_DELTA; | 89 | static int compute = COMPUTE_DELTA_ABS; |
90 | 90 | ||
91 | static int compute_2_hpp[COMPUTE_MAX] = { | 91 | static int compute_2_hpp[COMPUTE_MAX] = { |
92 | [COMPUTE_DELTA] = PERF_HPP_DIFF__DELTA, | 92 | [COMPUTE_DELTA] = PERF_HPP_DIFF__DELTA, |
@@ -810,7 +810,7 @@ static const struct option options[] = { | |||
810 | OPT_BOOLEAN('b', "baseline-only", &show_baseline_only, | 810 | OPT_BOOLEAN('b', "baseline-only", &show_baseline_only, |
811 | "Show only items with match in baseline"), | 811 | "Show only items with match in baseline"), |
812 | OPT_CALLBACK('c', "compute", &compute, | 812 | OPT_CALLBACK('c', "compute", &compute, |
813 | "delta,delta-abs,ratio,wdiff:w1,w2 (default delta)", | 813 | "delta,delta-abs,ratio,wdiff:w1,w2 (default delta-abs)", |
814 | "Entries differential computation selection", | 814 | "Entries differential computation selection", |
815 | setup_compute), | 815 | setup_compute), |
816 | OPT_BOOLEAN('p', "period", &show_period, | 816 | OPT_BOOLEAN('p', "period", &show_period, |