linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jiri Olsa <jolsa@redhat.com>
To: linux-kernel@vger.kernel.org
Cc: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Ingo Molnar <mingo@elte.hu>, Paul Mackerras <paulus@samba.org>,
	Corey Ashford <cjashfor@linux.vnet.ibm.com>,
	Frederic Weisbecker <fweisbec@gmail.com>,
	Namhyung Kim <namhyung@kernel.org>
Subject: [PATCH/RFC 00/14] perf, tool: Fix endian issues
Date: Wed, 28 Nov 2012 14:52:35 +0100	[thread overview]
Message-ID: <1354110769-2998-1-git-send-email-jolsa@redhat.com> (raw)

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(-)

             reply	other threads:[~2012-11-28 13:57 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-28 13:52 Jiri Olsa [this message]
2012-11-28 13:52 ` [PATCH 01/14] perf tool: Introduce perf_hpp__list for period related columns Jiri Olsa
2012-11-29  7:44   ` Namhyung Kim
2012-11-29 10:52     ` Namhyung Kim
2012-11-28 13:52 ` [PATCH 02/14] perf tool: Add struct perf_hpp_fmt into hpp callbacks Jiri Olsa
2012-11-28 13:52 ` [PATCH 03/14] perf tool: Fix period symbol_conf.field_sep display Jiri Olsa
2012-11-29  7:56   ` Namhyung Kim
2012-11-29 11:53     ` Jiri Olsa
2012-11-29 13:40       ` [PATCH] perf hists: Fix period symbol_conf.field_sep display, again Jiri Olsa
2012-11-29 17:35         ` Arnaldo Carvalho de Melo
2012-11-28 13:52 ` [PATCH 04/14] perf diff: Remove displacement from struct hist_entry_diff Jiri Olsa
2012-11-29  8:00   ` Namhyung Kim
2013-01-24 18:50   ` [tip:perf/core] " tip-bot for Jiri Olsa
2012-11-28 13:52 ` [PATCH 05/14] perf diff: Change compute methods to work with pair directly Jiri Olsa
2012-11-29 11:15   ` Namhyung Kim
2013-01-24 18:51   ` [tip:perf/core] " tip-bot for Jiri Olsa
2012-11-28 13:52 ` [PATCH 06/14] perf diff: Change formula " Jiri Olsa
2012-11-29 11:15   ` Namhyung Kim
2013-01-24 18:52   ` [tip:perf/core] " tip-bot for Jiri Olsa
2012-11-28 13:52 ` [PATCH 07/14] perf diff: Add callback to hists__match/hists__link functions Jiri Olsa
2012-11-28 13:52 ` [PATCH 08/14] perf diff: Change diff command to work over multiple data files Jiri Olsa
2012-11-29 11:37   ` Namhyung Kim
2012-11-29 11:53     ` Jiri Olsa
2012-11-28 13:52 ` [PATCH 09/14] perf diff: Update perf diff documentation for multiple data comparison Jiri Olsa
2012-11-29 11:52   ` Namhyung Kim
2012-11-29 12:06     ` Jiri Olsa
2012-11-28 13:52 ` [PATCH 10/14] perf tool: Centralize default columns init in perf_hpp__init Jiri Olsa
2012-11-29 11:55   ` Namhyung Kim
2012-11-29 12:13     ` Jiri Olsa
2012-11-30  5:06       ` Namhyung Kim
2012-11-30 11:56         ` Jiri Olsa
2012-11-28 13:52 ` [PATCH 11/14] perf diff: Making compute functions static Jiri Olsa
2012-11-28 13:52 ` [PATCH 12/14] perf diff: Display data file info ahead of the diff output Jiri Olsa
2012-11-28 13:52 ` [PATCH 13/14] perf diff: Display zero calculation results Jiri Olsa
2012-11-28 13:52 ` [PATCH 14/14] perf diff: Add generic order option for compute sorting Jiri Olsa
2012-11-29 12:02   ` Namhyung Kim
2012-11-29 12:19     ` Jiri Olsa
2012-11-28 13:56 ` [PATCH/RFC 00/14] perf, diff: Support for multiple files Jiri Olsa
2012-11-28 15:57   ` Jiri Olsa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1354110769-2998-1-git-send-email-jolsa@redhat.com \
    --to=jolsa@redhat.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=acme@ghostprotocols.net \
    --cc=cjashfor@linux.vnet.ibm.com \
    --cc=fweisbec@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=namhyung@kernel.org \
    --cc=paulus@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).