hi, this is v2 of diff command changes proposed in here: https://lkml.org/lkml/2012/9/6/344
It's now rebased on new hists hpp* interface plus few more additional changes. I'm still testing/using it to come up with some proved usability, but would be nice to have it in anyway. v2 changes: - based on hpp* interface - patch 13/14 includes samples with no symbol in diff output makes diff numbers looks the same as in report - patch 14/14 displays empty space instead of zeros for non paired samples makes the output more clear Attached patches: 01/14 perf hists: Add struct hists pointer to struct hist_entry 02/14 perf diff: Refactor diff displacement possition info 03/14 perf hists: Separate overhead and baseline columns 04/14 perf tools: Removing hists pair argument from output path 05/14 perf diff: Add -b option for perf diff to display paired entries only 06/14 perf tool: Add hpp interface to enable/disable hpp column 07/14 perf diff: Add ratio computation way to compare hist entries 08/14 perf diff: Removing the total_period argument from output code 09/14 perf diff: Add option to sort entries based on diff computation 10/14 perf diff: Add weighted diff computation way to compare hist entries 11/14 perf diff: Add -p option to display period values for hist entries 12/14 perf diff: Add -F option to display formula for computation 13/14 perf diff: Include samples without symbol in overall stats 14/14 perf diff: Display empty space for non paired samples Available also at: git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git perf/diff Cc: Arnaldo Carvalho de Melo <a...@ghostprotocols.net> Cc: Peter Zijlstra <a.p.zijls...@chello.nl> Cc: Ingo Molnar <mi...@elte.hu> Cc: Paul Mackerras <pau...@samba.org> Cc: Corey Ashford <cjash...@linux.vnet.ibm.com> Cc: Frederic Weisbecker <fweis...@gmail.com> Cc: Paul E. McKenney <paul...@linux.vnet.ibm.com> Cc: Andi Kleen <a...@firstfloor.org> Cc: David Ahern <dsah...@gmail.com> Cc: Namhyung Kim <namhy...@kernel.org> --- tools/perf/Documentation/perf-diff.txt | 60 ++++++++++++ tools/perf/builtin-diff.c | 462 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----- tools/perf/builtin-report.c | 4 +- tools/perf/builtin-top.c | 2 +- tools/perf/ui/browsers/hists.c | 6 +- tools/perf/ui/gtk/browser.c | 6 +- tools/perf/ui/hist.c | 256 +++++++++++++++++++++++++++++++++++++------------ tools/perf/ui/setup.c | 2 +- tools/perf/ui/stdio/hist.c | 45 ++++----- tools/perf/util/hist.c | 2 + tools/perf/util/hist.h | 19 ++-- tools/perf/util/sort.h | 21 ++++- 12 files changed, 759 insertions(+), 126 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/