mbox

[00/32,v3] perf/core improvements and fixes

Message ID 1385667660-14159-1-git-send-email-acme@infradead.org
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-core-for-mingo

Message

Arnaldo Carvalho de Melo Nov. 28, 2013, 7:41 p.m. UTC
From: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>

Hi Ingo,

	Ingo, this is the 3rd try, this time removing the last two patches in
the series, from Jean, till we get the problems I found and the one reported by
Jiri sorted out.

	Jiri told me the problem exhibits when one doesn't have libunwind devel
files installed.

	At some point we're considering using things like the 'mock' tool to
do the tools/perf/tests/ builds tests with all possible development packages
combos to detect these kinds of problems earlier.

	Please consider applying,

Best Regards,

- Arnaldo

The following changes since commit 65661f96d3b32f4b28fef26d21be81d7e173b965:

  perf/x86: Add RAPL hrtimer support (2013-11-27 15:31:23 +0100)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-core-for-mingo

for you to fetch changes up to 26286141a43251bed548349efbf60b61f1c433e8:

  perf tools: Fix tags/TAGS targets rebuilding (2013-11-27 16:47:14 -0300)

----------------------------------------------------------------
perf/core improvements and fixes:

. Make per-cpu mmaps the default in 'perf record', from Adrian Hunter.

. Default -t/--thread 'perf record' option to no inheritance,
  from Adrian Hunter.

. Make 'perf top -g' refer to callchains, for consistency with other tools,
  from David Ahern.

. Skip ignored symbols while printing callchain, from David Ahern.

. Print callchains and symbols if they exist in 'perf script',
  from David Ahern.

. Remove thread summary coloring in 'perf trace', from Pekka Enberg.

. zsh completion support, from Ramkumar Ramachandra.

. 'perf timechart' improvements, including backtrace support,
  from Stanislav Fomichev.

. Fix using kcore files stored in the buildid cache when doing report/annotate
  in non-live sessions, from Adrian Hunter

. Minor 'timechart' cleanups.

. Fix tags/TAGS targets rebuilding, from Jiri Olsa.

. Add options to show comm, fork, exit and mmap PERF_RECORD_ events in
  'perf script', from Namhyung Kim.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

----------------------------------------------------------------
Adrian Hunter (5):
      perf record: Make per-cpu mmaps the default.
      perf tools: Allow '--inherit' as the negation of '--no-inherit'
      perf tools: Add option macro OPT_BOOLEAN_SET
      perf record: Default -t option to no inheritance
      perf symbols: Fix not finding kcore in buildid cache

Arnaldo Carvalho de Melo (2):
      perf timechart: Remove some needless struct forward declarations
      perf timechart: Remove misplaced __maybe_unused

David Ahern (6):
      perf top: Make -g refer to callchains
      perf evsel: Skip ignored symbols while printing callchain
      perf symbols: Move idle syms check from top to generic function
      perf thread: Move comm_list check into function
      perf tools: Export setup_list
      perf script: Print callchains and symbols if they exist

Jiri Olsa (1):
      perf tools: Fix tags/TAGS targets rebuilding

Namhyung Kim (3):
      perf script: Move evname print code to process_event()
      perf script: Print comm, fork and exit events also
      perf script: Print mmap[2] events also

Pekka Enberg (1):
      perf trace: Remove thread summary coloring

Ramkumar Ramachandra (5):
      perf completion: Introduce a layer of indirection
      perf completion: Factor out compgen stuff
      perf completion: Factor out call to __ltrim_colon_completions
      perf completion: Introduce zsh support
      perf completion: Rename file to reflect zsh support

Stanislav Fomichev (8):
      perf timechart: Always try to print at least 15 tasks
      perf timechart: Add option to limit number of tasks
      perf timechart: Use proc_num to implement --power-only
      perf timechart: Add support for displaying only tasks related data
      perf timechart: Group figures and add title with details
      perf timechart: Add support for -P and -T in timechart recording
      perf timechart: Add backtrace support
      perf timechart: dynamically determine event fields offset

Steven Rostedt (1):
      tools lib traceevent: Use helper trace-seq in print functions like kernel does

 tools/lib/traceevent/event-parse.c                 |   7 +-
 tools/perf/Documentation/perf-record.txt           |  12 +-
 tools/perf/Documentation/perf-script.txt           |   6 +
 tools/perf/Documentation/perf-timechart.txt        |  26 +-
 tools/perf/Documentation/perf-top.txt              |   5 +-
 tools/perf/Makefile                                |   7 +-
 tools/perf/Makefile.perf                           |   4 +-
 tools/perf/builtin-record.c                        |  13 +-
 tools/perf/builtin-script.c                        | 209 ++++++++++-
 tools/perf/builtin-timechart.c                     | 418 ++++++++++++++-------
 tools/perf/builtin-top.c                           |  29 +-
 tools/perf/builtin-trace.c                         |  13 +-
 tools/perf/{bash_completion => perf-completion.sh} | 100 ++++-
 tools/perf/perf.h                                  |   1 +
 tools/perf/tests/attr/test-record-no-inherit       |   2 +-
 tools/perf/util/event.c                            |   3 +-
 tools/perf/util/evlist.c                           |   6 +-
 tools/perf/util/evsel.c                            |   5 +-
 tools/perf/util/parse-options.c                    |  21 ++
 tools/perf/util/parse-options.h                    |   8 +
 tools/perf/util/session.c                          |  10 +-
 tools/perf/util/svghelper.c                        |  77 +++-
 tools/perf/util/svghelper.h                        |  11 +-
 tools/perf/util/symbol.c                           |  41 +-
 tools/perf/util/symbol.h                           |   4 +
 tools/perf/util/target.c                           |  11 +-
 tools/perf/util/target.h                           |   4 +-
 tools/perf/util/thread.h                           |  12 +
 28 files changed, 831 insertions(+), 234 deletions(-)
 rename tools/perf/{bash_completion => perf-completion.sh} (65%)

Comments

Ingo Molnar Nov. 30, 2013, 12:45 p.m. UTC | #1
* Arnaldo Carvalho de Melo <acme@infradead.org> wrote:

> From: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
> 
> Hi Ingo,
> 
> 	Ingo, this is the 3rd try, this time removing the last two patches in
> the series, from Jean, till we get the problems I found and the one reported by
> Jiri sorted out.
> 
> 	Jiri told me the problem exhibits when one doesn't have libunwind devel
> files installed.
> 
> 	At some point we're considering using things like the 'mock' tool to
> do the tools/perf/tests/ builds tests with all possible development packages
> combos to detect these kinds of problems earlier.
> 
> 	Please consider applying,
> 
> Best Regards,
> 
> - Arnaldo
> 
> The following changes since commit 65661f96d3b32f4b28fef26d21be81d7e173b965:
> 
>   perf/x86: Add RAPL hrtimer support (2013-11-27 15:31:23 +0100)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-core-for-mingo
> 
> for you to fetch changes up to 26286141a43251bed548349efbf60b61f1c433e8:
> 
>   perf tools: Fix tags/TAGS targets rebuilding (2013-11-27 16:47:14 -0300)
> 
> ----------------------------------------------------------------
> perf/core improvements and fixes:
> 
> . Make per-cpu mmaps the default in 'perf record', from Adrian Hunter.
> 
> . Default -t/--thread 'perf record' option to no inheritance,
>   from Adrian Hunter.
> 
> . Make 'perf top -g' refer to callchains, for consistency with other tools,
>   from David Ahern.
> 
> . Skip ignored symbols while printing callchain, from David Ahern.
> 
> . Print callchains and symbols if they exist in 'perf script',
>   from David Ahern.
> 
> . Remove thread summary coloring in 'perf trace', from Pekka Enberg.
> 
> . zsh completion support, from Ramkumar Ramachandra.
> 
> . 'perf timechart' improvements, including backtrace support,
>   from Stanislav Fomichev.
> 
> . Fix using kcore files stored in the buildid cache when doing report/annotate
>   in non-live sessions, from Adrian Hunter
> 
> . Minor 'timechart' cleanups.
> 
> . Fix tags/TAGS targets rebuilding, from Jiri Olsa.
> 
> . Add options to show comm, fork, exit and mmap PERF_RECORD_ events in
>   'perf script', from Namhyung Kim.
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> ----------------------------------------------------------------
> Adrian Hunter (5):
>       perf record: Make per-cpu mmaps the default.
>       perf tools: Allow '--inherit' as the negation of '--no-inherit'
>       perf tools: Add option macro OPT_BOOLEAN_SET
>       perf record: Default -t option to no inheritance
>       perf symbols: Fix not finding kcore in buildid cache
> 
> Arnaldo Carvalho de Melo (2):
>       perf timechart: Remove some needless struct forward declarations
>       perf timechart: Remove misplaced __maybe_unused
> 
> David Ahern (6):
>       perf top: Make -g refer to callchains
>       perf evsel: Skip ignored symbols while printing callchain
>       perf symbols: Move idle syms check from top to generic function
>       perf thread: Move comm_list check into function
>       perf tools: Export setup_list
>       perf script: Print callchains and symbols if they exist
> 
> Jiri Olsa (1):
>       perf tools: Fix tags/TAGS targets rebuilding
> 
> Namhyung Kim (3):
>       perf script: Move evname print code to process_event()
>       perf script: Print comm, fork and exit events also
>       perf script: Print mmap[2] events also
> 
> Pekka Enberg (1):
>       perf trace: Remove thread summary coloring
> 
> Ramkumar Ramachandra (5):
>       perf completion: Introduce a layer of indirection
>       perf completion: Factor out compgen stuff
>       perf completion: Factor out call to __ltrim_colon_completions
>       perf completion: Introduce zsh support
>       perf completion: Rename file to reflect zsh support
> 
> Stanislav Fomichev (8):
>       perf timechart: Always try to print at least 15 tasks
>       perf timechart: Add option to limit number of tasks
>       perf timechart: Use proc_num to implement --power-only
>       perf timechart: Add support for displaying only tasks related data
>       perf timechart: Group figures and add title with details
>       perf timechart: Add support for -P and -T in timechart recording
>       perf timechart: Add backtrace support
>       perf timechart: dynamically determine event fields offset
> 
> Steven Rostedt (1):
>       tools lib traceevent: Use helper trace-seq in print functions like kernel does
> 
>  tools/lib/traceevent/event-parse.c                 |   7 +-
>  tools/perf/Documentation/perf-record.txt           |  12 +-
>  tools/perf/Documentation/perf-script.txt           |   6 +
>  tools/perf/Documentation/perf-timechart.txt        |  26 +-
>  tools/perf/Documentation/perf-top.txt              |   5 +-
>  tools/perf/Makefile                                |   7 +-
>  tools/perf/Makefile.perf                           |   4 +-
>  tools/perf/builtin-record.c                        |  13 +-
>  tools/perf/builtin-script.c                        | 209 ++++++++++-
>  tools/perf/builtin-timechart.c                     | 418 ++++++++++++++-------
>  tools/perf/builtin-top.c                           |  29 +-
>  tools/perf/builtin-trace.c                         |  13 +-
>  tools/perf/{bash_completion => perf-completion.sh} | 100 ++++-
>  tools/perf/perf.h                                  |   1 +
>  tools/perf/tests/attr/test-record-no-inherit       |   2 +-
>  tools/perf/util/event.c                            |   3 +-
>  tools/perf/util/evlist.c                           |   6 +-
>  tools/perf/util/evsel.c                            |   5 +-
>  tools/perf/util/parse-options.c                    |  21 ++
>  tools/perf/util/parse-options.h                    |   8 +
>  tools/perf/util/session.c                          |  10 +-
>  tools/perf/util/svghelper.c                        |  77 +++-
>  tools/perf/util/svghelper.h                        |  11 +-
>  tools/perf/util/symbol.c                           |  41 +-
>  tools/perf/util/symbol.h                           |   4 +
>  tools/perf/util/target.c                           |  11 +-
>  tools/perf/util/target.h                           |   4 +-
>  tools/perf/util/thread.h                           |  12 +
>  28 files changed, 831 insertions(+), 234 deletions(-)
>  rename tools/perf/{bash_completion => perf-completion.sh} (65%)

Pulled, thanks a lot Arnaldo!

	Ingo