Messages in this thread | | | Date | Wed, 28 Nov 2012 14:56:55 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH/RFC 00/14] perf, diff: Support for multiple files |
| |
aargh.. fixed subject.. copy&paste error ;-)
also patches are available here:
git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git perf/multi2
thanks, jirka
On Wed, Nov 28, 2012 at 02:52:35PM +0100, Jiri Olsa wrote: > hi, > adding support to display diff for more than 2 perf.data files. > Sending as RFC, since the change touches lot of hists code, > so I might be breaking something I missed.. still testing. > > Also it could colide with Namhyung changes for group report > wrt patches 7 and 8, where I changed the linking of matching > hists entries. > > The doc was updated with info about current perf diff processing. > > Attached patches: > 01/14 perf tool: Introduce perf_hpp__list for period related columns > 02/14 perf tool: Add struct perf_hpp_fmt into hpp callbacks > 03/14 perf tool: Fix period symbol_conf.field_sep display > 04/14 perf diff: Remove displacement from struct hist_entry_diff > 05/14 perf diff: Change compute methods to work with pair directly > 06/14 perf diff: Change formula methods to work with pair directly > 07/14 perf diff: Add callback to hists__match/hists__link functions > 08/14 perf diff: Change diff command to work over multiple data files > 09/14 perf diff: Update perf diff documentation for multiple data comparison > 10/14 perf tool: Centralize default columns init in perf_hpp__init > 11/14 perf diff: Making compute functions static > 12/14 perf diff: Display data file info ahead of the diff output > 13/14 perf diff: Display zero calculation results > 14/14 perf diff: Add generic order option for compute sorting > > Example of multiple perf diff output: > > $ perf diff perf.data.[123456] > # Event 'cycles:u' > # > # Data files: > # [0] perf.data.1 > # [1] perf.data.2 > # [2] perf.data.3 > # [3] perf.data.4 > # [4] perf.data.5 > # [5] perf.data.6 > # > # Baseline/0 Delta/1 Delta/2 Delta/3 Delta/4 Delta/5 Shared Object Symbol > # .......... ....... ....... ....... ....... ....... ................. ......................... > # > +73.05% [kernel.kallsyms] [k] page_fault > +26.16% ld-2.15.so [.] _dl_next_ld_env_entry > +15.40% ld-2.15.so [.] _dl_sysdep_start > 71.48% -26.52% libc-2.15.so [.] __strcmp_sse2 > 27.69% -1.65% -4.83% -4.68% ld-2.15.so [.] dl_main > 0.82% -0.05% -0.34% -0.15% -0.13% -0.03% ld-2.15.so [.] _start > +39.15% libc-2.15.so [.] error_tail > +73.18% libc-2.15.so [.] __strcasecmp_l_sse2 > +76.30% libc-2.15.so [.] __stpcpy_sse2 > +76.46% libc-2.15.so [.] _IO_getline_info > > thanks, > jirka > > Cc: Arnaldo Carvalho de Melo <acme@ghostprotocols.net> > Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> > Cc: Ingo Molnar <mingo@elte.hu> > 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> > --- > tools/perf/Documentation/perf-diff.txt | 77 ++++++++-- > tools/perf/builtin-diff.c | 747 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++------------------- > tools/perf/ui/browsers/hists.c | 22 +-- > tools/perf/ui/gtk/browser.c | 26 +--- > tools/perf/ui/hist.c | 392 ++++++++++++++--------------------------------- > tools/perf/ui/stdio/hist.c | 17 +-- > tools/perf/util/hist.c | 29 ++-- > tools/perf/util/hist.h | 42 ++--- > tools/perf/util/sort.h | 38 ++--- > 9 files changed, 855 insertions(+), 535 deletions(-)
| |