linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

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