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>,
Andi Kleen <ak@linux.intel.com>, Ben Cheng <bccheng@google.com>,
Corey Ashford <cjashfor@linux.vnet.ibm.com>,
David Ahern <dsahern@gmail.com>,
Dongsheng Yang <yangds.fnst@cn.fujitsu.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.kim@lge.com>,
patches@linaro.org, Paul Mackerras <paulus@samba.org>,
Peter Zijlstra <peterz@infradead.org>,
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: [GIT PULL 00/34] perf/core improvements and fixes
Date: Wed, 11 Dec 2013 12:01:01 +0100 [thread overview]
Message-ID: <20131211110101.GA25541@gmail.com> (raw)
In-Reply-To: <1386707476-12370-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,
>
> The first 20 patches in this series are the same as in the previous
> one, so I'm not reposting them now.
>
> This series has the code style/constification changes you suggested in
> the symbols code and then a reworked fix to the basename problem plus some
> more patches not present in the previous series.
>
> Please let me know if you find any further problems,
>
> Best Regards,
>
> - Arnaldo
>
> The following changes since commit 6d65894bc028d0342829ea1e64c9e9efad571124:
>
> tools lib traceevent: Update kvm plugin with is_writable_pte helper (2013-12-04 15:38:14 -0300)
>
> 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 1448fef40af6079de38380c3a81bcf9994a1037d:
>
> perf unwinding: Use the per-feature check flags (2013-12-10 16:51:12 -0300)
>
> ----------------------------------------------------------------
> perf/core improvements and fixes:
>
> . Add an option in 'perf script' to print the source line number, from Adrian Hunter
>
> . Add --header/--header-only options to 'script' and 'report', the default is not
> tho show the header info, but as this has been the default for some time,
> leave a single line explaining how to obtain that information, from Jiri Olsa.
>
> . Fix symoff printing in callchains in 'perf script', from Adrian Hunter.
>
> . Assorted mmap_pages handling fixes, from Adrian Hunter.
>
> . Fix summary percentage when processing files in 'perf trace', fom David Ahern.
>
> . Handle old kernels where the "raw_syscalls" tracepoints were called plan "syscalls",
> in 'perf trace', from David Ahern.
>
> . Several man pages typo fixes from Dongsheng Yang.
>
> . Add '-v' option to 'perf kvm', from Dongsheng Yang.
>
> . Make perf kvm diff support --guestmount, from Dongsheng Yang.
>
> . Get rid of several die() calls in libtraceevent, from Namhyung Kim.
>
> . Use basename() in a more robust way, to avoid problems related to different
> system library implementations for that function, from Stephane Eranian.
>
> . Remove open coded management of short_name_allocated member, from Adrian Hunter
>
> . Several cleanups in the "dso" methods, constifying some parameters and
> renaming some fields to clarify its purpose.
>
> . Add per-feature check flags, fixing libunwind related build problems on some
> architectures, from Jean Pihet.
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>
> ----------------------------------------------------------------
> Adrian Hunter (7):
> perf script: Fix symoff printing in callchains
> perf script: Add an option to print the source line number
> perf record: Fix display of incorrect mmap pages
> perf evlist: Remove unnecessary parentheses
> perf evlist: Fix max mmap_pages
> perf evlist: Fix mmap pages rounding to power of 2
> perf symbols: Remove open coded management of short_name_allocated member
>
> Arnaldo Carvalho de Melo (8):
> perf symbols: Rename [sl]name_alloc to match the members they refer to
> perf machine: Don't open code assign dso->short_name
> perf symbols: Set alloc flag close to setting the long_name
> perf symbols: Remove open coded management of long_name_allocated member
> perf symbols: Constify dso->long_name
> perf symbols: Set freed members to NULL in dso destructor
> perf symbols: Constify some DSO methods parameters
> perf symbols: Rename filename argument
>
> David Ahern (2):
> perf trace: Add support for syscalls vs raw_syscalls
> perf trace: Fix summary percentage when processing files
>
> Dongsheng Yang (6):
> perf kvm: Introduce option -v for perf kvm command.
> perf kvm: Fix bug in 'stat report'
> perf archive: Remove duplicated 'runs' in man page
> perf annotate: Fix typo
> perf kvm: Move code to generate filename for perf-kvm to function.
> perf kvm: Make perf kvm diff support --guestmount.
>
> Jean Pihet (2):
> perf tools: Add per-feature check flags
> perf unwinding: Use the per-feature check flags
>
> Jiri Olsa (2):
> perf report: Add --header/--header-only options
> perf script: Add --header/--header-only options
>
> Namhyung Kim (5):
> tools lib traceevent: Get rid of malloc_or_die() in pevent_filter_alloc()
> tools lib traceevent: Get rid of malloc_or_die() in add_event()
> tools lib traceevent: Get rid of die() in create_arg_item()
> tools lib traceevent: Get rid of malloc_or_die() in pevent_filter_add_filter_str()
> tools lib traceevent: Get rid of die() in pevent_filter_clear_trivial()
>
> Stephane Eranian (1):
> perf symbols: Fix bug in usage of the basename() function
>
> Steven Rostedt (1):
> tools lib traceevent: Report better error message on bad function args
>
> tools/lib/traceevent/event-parse.c | 28 +++++---
> tools/lib/traceevent/event-parse.h | 2 +-
> tools/lib/traceevent/parse-filter.c | 57 +++++++++++----
> tools/perf/Documentation/perf-archive.txt | 6 +-
> tools/perf/Documentation/perf-kvm.txt | 7 +-
> tools/perf/Documentation/perf-report.txt | 9 +++
> tools/perf/Documentation/perf-script.txt | 8 ++-
> tools/perf/builtin-annotate.c | 2 +-
> tools/perf/builtin-diff.c | 3 +-
> tools/perf/builtin-kvm.c | 11 ++-
> tools/perf/builtin-record.c | 2 +-
> tools/perf/builtin-report.c | 22 +++++-
> tools/perf/builtin-script.c | 23 +++++-
> tools/perf/builtin-trace.c | 32 ++++++++-
> tools/perf/config/Makefile | 52 ++++++++------
> tools/perf/config/feature-checks/Makefile | 8 +--
> tools/perf/util/annotate.c | 2 +-
> tools/perf/util/build-id.c | 2 +-
> tools/perf/util/build-id.h | 2 +-
> tools/perf/util/dso.c | 112 ++++++++++++++++++++----------
> tools/perf/util/dso.h | 16 ++---
> tools/perf/util/evlist.c | 10 +--
> tools/perf/util/header.c | 6 +-
> tools/perf/util/machine.c | 6 +-
> tools/perf/util/map.c | 17 +++++
> tools/perf/util/map.h | 2 +
> tools/perf/util/probe-event.c | 2 +-
> tools/perf/util/session.c | 15 +++-
> tools/perf/util/session.h | 1 +
> tools/perf/util/srcline.c | 2 +-
> tools/perf/util/symbol.c | 38 ++++------
> tools/perf/util/symbol.h | 3 +-
> tools/perf/util/util.c | 14 ++++
> tools/perf/util/util.h | 14 ++++
> tools/perf/util/vdso.c | 2 +-
> 35 files changed, 375 insertions(+), 163 deletions(-)
Pulled, thanks a lot Arnaldo!
Ingo
next prev parent reply other threads:[~2013-12-11 11:01 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-10 20:31 [GIT PULL 00/34] perf/core improvements and fixes Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 21/34 v2] perf report: Add --header/--header-only options Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 22/34] perf script: " Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 23/34] perf symbols: Rename [sl]name_alloc to match the members they refer to Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 24/34] perf machine: Don't open code assign dso->short_name Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 25/34] perf symbols: Remove open coded management of short_name_allocated member Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 26/34] perf symbols: Set alloc flag close to setting the long_name Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 27/34] perf symbols: Remove open coded management of long_name_allocated member Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 28/34] perf symbols: Constify dso->long_name Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 29/34] perf symbols: Set freed members to NULL in dso destructor Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 30/34] perf symbols: Constify some DSO methods parameters Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 31/34] perf symbols: Rename filename argument Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 32/34 v3] perf symbols: Fix bug in usage of the basename() function Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 33/34] perf tools: Add per-feature check flags Arnaldo Carvalho de Melo
2013-12-10 20:31 ` [PATCH 34/34] perf unwinding: Use the " Arnaldo Carvalho de Melo
2013-12-11 11:01 ` Ingo Molnar [this message]
-- strict thread matches above, loose matches on Subject: below --
2017-12-14 17:42 [GIT PULL 00/34] perf/core improvements and fixes Arnaldo Carvalho de Melo
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=20131211110101.GA25541@gmail.com \
--to=mingo@kernel.org \
--cc=acme@ghostprotocols.net \
--cc=acme@infradead.org \
--cc=acme@redhat.com \
--cc=adrian.hunter@intel.com \
--cc=ak@linux.intel.com \
--cc=bccheng@google.com \
--cc=cjashfor@linux.vnet.ibm.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.kim@lge.com \
--cc=patches@linaro.org \
--cc=paulus@samba.org \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=will.deacon@arm.com \
--cc=yangds.fnst@cn.fujitsu.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;
as well as URLs for NNTP newsgroup(s).