hi, this is v3 of diff command changes proposed in here: https://lkml.org/lkml/2012/9/6/344
Half of the previous patchset version (hists related) got already in together with changes of Namhyung Kim. This is what remained, plus additional small fixies from review. - small delta format changes (Namhyung Kim) - better error messages for -c option (Andi Kleen) Thanks a lot to Andi Kleen, who tested this and provided great feedback! I plan to address following in future: - bucketizing per line instead of per function - allow diff to specify vmlinux files per data - recognize/compare same functions within 2 different kernels - define groups of callers that have a combined cost including all callees, use this as the diff unit (http://marc.info/?l=linux-kernel&m=134919541904105&w=2) Attached patches: 1/8 perf diff: Add -b option for perf diff to display paired entries only 2/8 perf diff: Add ratio computation way to compare hist entries 3/8 perf diff: Add option to sort entries based on diff computation 4/8 perf diff: Add weighted diff computation way to compare hist entries 5/8 perf diff: Add -p option to display period values for hist entries 6/8 perf diff: Add -F option to display formula for computation 7/8 perf diff: Include samples without symbol in overall stats 8/8 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/diff1 wbr, jirka 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 | 401 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-- tools/perf/ui/hist.c | 134 ++++++++++++++++++++++++++---- tools/perf/ui/stdio/hist.c | 2 +- tools/perf/util/hist.h | 8 ++ tools/perf/util/sort.h | 18 ++++ 6 files changed, 600 insertions(+), 23 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/
