mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@kernel.org>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: linux-kernel@vger.kernel.org,
	Adrian Hunter <adrian.hunter@intel.com>,
	Alexander Shishkin <alexander.shishkin@linux.intel.com>,
	Alexei Starovoitov <ast@kernel.org>,
	Andi Kleen <ak@linux.intel.com>, Cody P Schafer <dev@codyps.com>,
	David Ahern <dsahern@gmail.com>, He Kuang <hekuang@huawei.com>,
	Hemant Kumar <hemant@linux.vnet.ibm.com>,
	Jakub Jelen <jjelen@redhat.com>,
	Jan Stancek <jstancek@redhat.com>, Jiri Olsa <jolsa@redhat.com>,
	Josh Poimboeuf <jpoimboe@redhat.com>,
	Kan Liang <kan.liang@intel.com>, Kirill Smelkov <kirr@nexedi.com>,
	Li Zefan <lizefan@huawei.com>,
	Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>,
	Namhyung Kim <namhyung@kernel.org>,
	"Naveen N . Rao" <naveen.n.rao@linux.vnet.ibm.com>,
	Peter Zijlstra <peterz@infradead.org>,
	pi3orama@163.com,
	Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>,
	Wang Nan <wangnan0@huawei.com>,
	Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [GIT PULL 00/23] perf/core improvements and fixes
Date: Thu, 24 Mar 2016 08:32:12 +0100	[thread overview]
Message-ID: <20160324073212.GA15260@gmail.com> (raw)
In-Reply-To: <1458767880-3288-1-git-send-email-acme@kernel.org>


* Arnaldo Carvalho de Melo <acme@kernel.org> wrote:

> Hi Ingo,
> 
> 	Please consider pulling, test built on:
> 
>   [root@jouet ~]# dm
>   alldeps-debian: Ok
>   alldeps-mageia: Ok
>   alldeps-rhel7: Ok
>   alldeps-centos: Ok
>   alldeps-opensuse: Ok
>   alldeps-ubuntu: Ok
>   [root@jouet ~]# 
> 
> 	All the latest distro releases, updated (dnf/yum/apt-get/urpmi/zypper/etc
> upgraded as of a few days ago) as available on docker.io/vendors.
> 
> - Arnaldo
> 
> The following changes since commit ced30bc9129777d715057d06fc8dbdfd3b81e94d:
> 
>   Merge tag 'perf-core-for-mingo-20160310' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core (2016-03-11 09:40:25 +0100)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo-20160323
> 
> for you to fetch changes up to 76267147f233f347dbd8f4f8c05492e5ea0a7dce:
> 
>   perf llvm: Use strerror_r instead of the thread unsafe strerror one (2016-03-23 17:42:21 -0300)
> 
> ----------------------------------------------------------------
> perf/core improvements and fixes:
> 
> User visible:
> 
> - Fix documentation of :ppp modifier in 'perf list' (Andi Kleen)
> 
> - Fix silly nodes bitfield bits/bytes length assertion in 'perf bench numa' (Jakub Jelen)
> 
> - Remove redundant CPU output in libtraceevent (Steven Rostedt)
> 
> - Remove 'core_id' check in topology 'perf test' (Sukadev Bhattiprolu)
> 
> Infrastructure:
> 
> - Record text offset in dso to calculate objdump address, to use with
>   modules in addition to vDSO symbol address calculations (Wang Nan)
> 
> - Move utilities.mak from perf to tools/scripts/ (Arnaldo Carvalho de Melo)
> 
> - Add cpumode to the perf_sample struct, this way we don't need to pass
>   the union event to the machine and thread resolving routines, shortening
>   function signatures and allowing the future introduction of a way
>   to use tracepoint events instead of the unavailable HW cycles counter on
>   powerpc guests in perf kvm by just hooking on perf_evsel__parse_sample,
>   at the end (Arnaldo Carvalho de Melo)
> 
> - Remove/unexport die() related infrastructure, that at some point will
>   finally be removed (Arnaldo Carvalho de Melo)
> 
> - Adopt linux/stringify.h from the kernel sources, not to touch this
>   kernel header from tools/ (Arnaldo Carvalho de Melo)
> 
> - Stop using strbuf for things we can instead trivially use libc's asprintf()
>   (Arnaldo Carvalho de Melo)
> 
> - Ditch tools/lib/util/abspath.c, its only exported function was used at just
>   one place and can be replaced by libc's realpath() (Arnaldo Carvalho de Melo)
> 
> - Use strerror_r() in the llvm infrastructure, tread safe, its what is used
>   elsewhere in tools/perf/ (Arnaldo Carvalho de Melo)
> 
> Cleanups:
> 
> - Removed misplaced or needless __maybe_unused/export (Arnaldo Carvalho de Melo)
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> ----------------------------------------------------------------
> Andi Kleen (1):
>       perf list: Fix documentation of :ppp
> 
> Arnaldo Carvalho de Melo (18):
>       tools: Move utilities.mak from perf to tools/scripts/
>       perf tools: Remove misplaced __maybe_unused
>       perf tests: Forward the perf_sample in the dwarf unwind test
>       perf tools: Add cpumode to struct perf_sample
>       perf machine: Rename perf_event__preprocess_sample to machine__resolve
>       perf thread: Rename perf_event__preprocess_sample_addr to thread__resolve
>       perf script: Remove lots of unused arguments
>       perf tools: Remove unused DIE_IF macro
>       perf tools: Simplify die() mechanism
>       perf tools: Remove needless 'extern' from function prototypes
>       tools include: Copy linux/stringify.h from the kernel
>       perf tools: Do not include stringify.h from the kernel sources
>       perf tools: Remove unused perf_pathdup, xstrdup functions
>       perf help: Use asprintf instead of adhoc equivalents
>       perf probe: No need to use formatting strbuf method
>       perf tools: Unexport some methods unused outside strbuf.c
>       perf llvm: Use realpath to canonicalize paths
>       perf llvm: Use strerror_r instead of the thread unsafe strerror one
> 
> Jakub Jelen (1):
>       perf bench numa: Fix assertion for nodes bitfield
> 
> Steven Rostedt (1):
>       tools lib traceevent: Remove redundant CPU output
> 
> Sukadev Bhattiprolu (1):
>       perf test: Remove 'core_id' check in topo test
> 
> Wang Nan (1):
>       perf symbols: Record text offset in dso to calculate objdump address
> 
>  tools/include/linux/stringify.h              | 12 +++++
>  tools/lib/api/Makefile                       |  2 +-
>  tools/lib/subcmd/Makefile                    |  2 +-
>  tools/lib/traceevent/event-parse.c           |  4 +-
>  tools/perf/Documentation/Makefile            |  2 +-
>  tools/perf/Documentation/perf-list.txt       |  6 ++-
>  tools/perf/Makefile.perf                     |  2 +-
>  tools/perf/arch/powerpc/util/header.c        |  4 +-
>  tools/perf/bench/bench.h                     | 22 ++++-----
>  tools/perf/bench/mem-memcpy-arch.h           |  2 +-
>  tools/perf/bench/mem-memset-arch.h           |  2 +-
>  tools/perf/bench/numa.c                      |  2 +-
>  tools/perf/builtin-annotate.c                |  2 +-
>  tools/perf/builtin-diff.c                    |  2 +-
>  tools/perf/builtin-help.c                    | 69 ++++++++++++--------------
>  tools/perf/builtin-inject.c                  |  8 +---
>  tools/perf/builtin-mem.c                     |  2 +-
>  tools/perf/builtin-report.c                  |  3 +-
>  tools/perf/builtin-script.c                  | 46 +++++++-----------
>  tools/perf/builtin-timechart.c               |  2 +-
>  tools/perf/builtin-top.c                     |  8 ++--
>  tools/perf/builtin-trace.c                   |  9 ++--
>  tools/perf/builtin.h                         | 64 ++++++++++++-------------
>  tools/perf/config/Makefile                   |  2 +-
>  tools/perf/tests/code-reading.c              |  5 +-
>  tools/perf/tests/dwarf-unwind.c              |  4 +-
>  tools/perf/tests/hists_common.c              |  6 ++-
>  tools/perf/tests/hists_cumulate.c            |  9 +---
>  tools/perf/tests/hists_filter.c              |  9 +---
>  tools/perf/tests/hists_link.c                | 20 ++------
>  tools/perf/tests/hists_output.c              |  9 +---
>  tools/perf/ui/gtk/hists.c                    |  2 +-
>  tools/perf/util/Build                        |  1 -
>  tools/perf/util/abspath.c                    | 37 --------------
>  tools/perf/util/annotate.h                   |  2 +-
>  tools/perf/util/auxtrace.h                   |  2 +-
>  tools/perf/util/build-id.c                   |  3 +-
>  tools/perf/util/cache.h                      | 24 ++++------
>  tools/perf/util/callchain.h                  |  4 +-
>  tools/perf/util/cgroup.h                     |  4 +-
>  tools/perf/util/cloexec.h                    |  2 +-
>  tools/perf/util/data-convert-bt.c            |  2 +-
>  tools/perf/util/db-export.c                  |  2 +-
>  tools/perf/util/dso.h                        |  3 +-
>  tools/perf/util/dwarf-aux.c                  | 10 ++--
>  tools/perf/util/dwarf-aux.h                  | 72 ++++++++++++++--------------
>  tools/perf/util/event.c                      | 23 ++++-----
>  tools/perf/util/event.h                      | 13 ++---
>  tools/perf/util/evsel.c                      |  1 +
>  tools/perf/util/genelf.h                     |  8 ++--
>  tools/perf/util/header.c                     |  5 --
>  tools/perf/util/header.h                     |  2 +-
>  tools/perf/util/hist.c                       |  2 +-
>  tools/perf/util/hist.h                       |  3 +-
>  tools/perf/util/intel-bts.c                  |  2 +-
>  tools/perf/util/jit.h                        | 12 ++---
>  tools/perf/util/llvm-utils.c                 | 24 ++++++----
>  tools/perf/util/llvm-utils.h                 |  7 ++-
>  tools/perf/util/machine.c                    | 14 +++---
>  tools/perf/util/machine.h                    |  2 +-
>  tools/perf/util/parse-events.h               | 21 ++++----
>  tools/perf/util/path.c                       | 30 ------------
>  tools/perf/util/probe-event.c                |  2 +-
>  tools/perf/util/probe-event.h                | 57 ++++++++++------------
>  tools/perf/util/probe-finder.c               |  8 ++--
>  tools/perf/util/probe-finder.h               | 24 +++++-----
>  tools/perf/util/quote.h                      |  2 +-
>  tools/perf/util/session.c                    |  5 +-
>  tools/perf/util/sort.c                       |  2 +-
>  tools/perf/util/stat-shadow.c                | 18 ++++---
>  tools/perf/util/strbuf.c                     |  9 +++-
>  tools/perf/util/strbuf.h                     | 21 ++++----
>  tools/perf/util/svghelper.h                  | 51 ++++++++++----------
>  tools/perf/util/symbol-elf.c                 | 12 ++---
>  tools/perf/util/symbol.h                     |  4 +-
>  tools/perf/util/usage.c                      |  8 +---
>  tools/perf/util/util.h                       | 27 ++++-------
>  tools/perf/util/wrapper.c                    | 12 -----
>  tools/{perf/config => scripts}/utilities.mak |  0
>  79 files changed, 385 insertions(+), 557 deletions(-)
>  create mode 100644 tools/include/linux/stringify.h
>  delete mode 100644 tools/perf/util/abspath.c
>  rename tools/{perf/config => scripts}/utilities.mak (100%)

Pulled, thanks a lot Arnaldo!

	Ingo

  parent reply	other threads:[~2016-03-24  7:32 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-23 21:17 Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 01/23] perf test: Remove 'core_id' check in topo test Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 02/23] tools: Move utilities.mak from perf to tools/scripts/ Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 03/23] perf symbols: Record text offset in dso to calculate objdump address Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 04/23] perf bench numa: Fix assertion for nodes bitfield Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 05/23] perf list: Fix documentation of :ppp Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 06/23] perf tools: Remove misplaced __maybe_unused Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 07/23] perf tests: Forward the perf_sample in the dwarf unwind test Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 08/23] perf tools: Add cpumode to struct perf_sample Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 09/23] perf machine: Rename perf_event__preprocess_sample to machine__resolve Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 10/23] perf thread: Rename perf_event__preprocess_sample_addr to thread__resolve Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 11/23] perf script: Remove lots of unused arguments Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 12/23] perf tools: Remove unused DIE_IF macro Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 13/23] perf tools: Simplify die() mechanism Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 14/23] perf tools: Remove needless 'extern' from function prototypes Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 15/23] tools lib traceevent: Remove redundant CPU output Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 16/23] tools include: Copy linux/stringify.h from the kernel Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 17/23] perf tools: Do not include stringify.h from the kernel sources Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 18/23] perf tools: Remove unused perf_pathdup, xstrdup functions Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 19/23] perf help: Use asprintf instead of adhoc equivalents Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 20/23] perf probe: No need to use formatting strbuf method Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 21/23] perf tools: Unexport some methods unused outside strbuf.c Arnaldo Carvalho de Melo
2016-03-23 21:17 ` [PATCH 22/23] perf llvm: Use realpath to canonicalize paths Arnaldo Carvalho de Melo
2016-03-23 21:18 ` [PATCH 23/23] perf llvm: Use strerror_r instead of the thread unsafe strerror one Arnaldo Carvalho de Melo
2016-03-24  7:32 ` Ingo Molnar [this message]
  -- strict thread matches above, loose matches on Subject: below --
2017-06-20  1:54 [GIT PULL 00/23] perf/core improvements and fixes Arnaldo Carvalho de Melo
2017-06-20  8:50 ` Ingo Molnar
2017-01-25 13:53 Arnaldo Carvalho de Melo
2015-10-19 21:39 Arnaldo Carvalho de Melo
2015-10-20  7:32 ` Ingo Molnar

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=20160324073212.GA15260@gmail.com \
    --to=mingo@kernel.org \
    --cc=acme@kernel.org \
    --cc=acme@redhat.com \
    --cc=adrian.hunter@intel.com \
    --cc=ak@linux.intel.com \
    --cc=alexander.shishkin@linux.intel.com \
    --cc=ast@kernel.org \
    --cc=dev@codyps.com \
    --cc=dsahern@gmail.com \
    --cc=hekuang@huawei.com \
    --cc=hemant@linux.vnet.ibm.com \
    --cc=jjelen@redhat.com \
    --cc=jolsa@redhat.com \
    --cc=jpoimboe@redhat.com \
    --cc=jstancek@redhat.com \
    --cc=kan.liang@intel.com \
    --cc=kirr@nexedi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizefan@huawei.com \
    --cc=masami.hiramatsu.pt@hitachi.com \
    --cc=namhyung@kernel.org \
    --cc=naveen.n.rao@linux.vnet.ibm.com \
    --cc=peterz@infradead.org \
    --cc=pi3orama@163.com \
    --cc=ravi.bangoria@linux.vnet.ibm.com \
    --cc=rostedt@goodmis.org \
    --cc=sukadev@linux.vnet.ibm.com \
    --cc=wangnan0@huawei.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

Powered by JetHome