From: Ingo Molnar <mingo@kernel.org>
To: Arnaldo Carvalho de Melo <acme@infradead.org>
Cc: linux-kernel@vger.kernel.org,
Arnaldo Carvalho de Melo <acme@ghostprotocols.net>,
Adrian Hunter <adrian.hunter@intel.com>,
Chia-I Wu <olvaffe@gmail.com>, David Ahern <dsahern@gmail.com>,
Frederic Weisbecker <fweisbec@gmail.com>,
Jean Pihet <jean.pihet@linaro.org>, Jiri Olsa <jolsa@redhat.com>,
linaro-kernel@lists.linaro.org, Mike Galbraith <efault@gmx.de>,
Namhyung Kim <namhyung@kernel.org>,
patches@linaro.org, Paul Mackerras <paulus@samba.org>,
Pekka Enberg <penberg@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Ramkumar Ramachandra <artagnon@gmail.com>,
Stanislav Fomichev <stfomichev@yandex-team.ru>,
Stephane Eranian <eranian@google.com>,
Steven Rostedt <rostedt@goodmis.org>,
Will Deacon <will.deacon@arm.com>,
Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [PATCH 00/32 v3] perf/core improvements and fixes
Date: Sat, 30 Nov 2013 13:45:21 +0100 [thread overview]
Message-ID: <20131130124521.GA12143@gmail.com> (raw)
In-Reply-To: <1385667660-14159-1-git-send-email-acme@infradead.org>
* 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
prev parent reply other threads:[~2013-11-30 12:45 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-28 19:41 [PATCH 00/32 v3] perf/core improvements and fixes Arnaldo Carvalho de Melo
2013-11-30 12:45 ` Ingo Molnar [this message]
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=20131130124521.GA12143@gmail.com \
--to=mingo@kernel.org \
--cc=acme@ghostprotocols.net \
--cc=acme@infradead.org \
--cc=acme@redhat.com \
--cc=adrian.hunter@intel.com \
--cc=artagnon@gmail.com \
--cc=dsahern@gmail.com \
--cc=efault@gmx.de \
--cc=eranian@google.com \
--cc=fweisbec@gmail.com \
--cc=jean.pihet@linaro.org \
--cc=jolsa@redhat.com \
--cc=linaro-kernel@lists.linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=namhyung@kernel.org \
--cc=olvaffe@gmail.com \
--cc=patches@linaro.org \
--cc=paulus@samba.org \
--cc=penberg@kernel.org \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=stfomichev@yandex-team.ru \
--cc=will.deacon@arm.com \
/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