Re: [PATCH/RFC 00/14] perf, diff: Support for multiple files

2012-11-28 Thread Jiri Olsa
On Wed, Nov 28, 2012 at 02:56:55PM +0100, Jiri Olsa wrote:
> 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

worked out some fixies with Arnaldo, new version is here:

git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git
perf/multi3

thanks,
jirka
--
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/


Re: [PATCH/RFC 00/14] perf, diff: Support for multiple files

2012-11-28 Thread Jiri Olsa
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 
> Cc: Peter Zijlstra 
> Cc: Ingo Molnar 
> Cc: Paul Mackerras 
> Cc: Corey Ashford 
> Cc: Frederic Weisbecker 
> Cc: Namhyung Kim 
> ---
>  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(-)
--
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/