From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751250AbcEGEwn (ORCPT ); Sat, 7 May 2016 00:52:43 -0400 Received: from mail-wm0-f46.google.com ([74.125.82.46]:37045 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750754AbcEGEwm (ORCPT ); Sat, 7 May 2016 00:52:42 -0400 Date: Sat, 7 May 2016 06:52:36 +0200 From: Ingo Molnar To: Arnaldo Carvalho de Melo Cc: linux-kernel@vger.kernel.org, Adrian Hunter , Andi Kleen , Chris Phlipot , David Ahern , Jiri Olsa , Milian Wolff , Namhyung Kim , Peter Zijlstra , Wang Nan , Arnaldo Carvalho de Melo Subject: Re: [GIT PULL 00/11] perf/core improvements and fixes Message-ID: <20160507045236.GA27659@gmail.com> References: <1462550912-15320-1-git-send-email-acme@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1462550912-15320-1-git-send-email-acme@kernel.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider pulling, > > - Arnaldo > > The following changes since commit c0edb7467c3d21b213ff734bfe810d81d2c6ed61: > > Merge tag 'perf-core-for-mingo-20160505' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core (2016-05-06 08:35:14 +0200) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo-20160506 > > for you to fetch changes up to d5d71e86d226abe7e08df5763127ed2bd07649a1: > > perf trace: Move futex_op beautifier to tools/perf/trace/beauty/ (2016-05-06 13:00:59 -0300) > > ---------------------------------------------------------------- > perf/core improvements and fixes: > > User visible: > > - Fix ordering of kernel/user entries in 'caller' mode, where the kernel and > user parts were being correctly inverted but kept in place wrt each other, > i.e. 'callee' (k1, k2, u3, u4) became 'caller' (k2, k1, u4, u3) when it > should be 'caller' (u4, u3, k2, k1) (Chris Phlipot) > > - In 'perf trace' don't print the raw arg syscall args for a syscall that has > no arguments, like gettid(). This was happening because just checking if > the syscall args list is NULL may mean that there are no args (e.g.: gettid) > or that there is no tracepoint info (e.g.: clone) (Arnaldo Carvalho de Melo) > > - Add extra output of counter values with 'perf stat -vv' (Andi Kleen) > > Infrastructure: > > - Expose callchain db export via the python API (Chris Phlipot) > > Code reorganization: > > - Move some more syscall arg beautifiers from the 'perf trace' main file to > separate files in tools/perf/trace/beauty/, to reduce the main file line > count (Arnaldo Carvalho de Melo) > > Signed-off-by: Arnaldo Carvalho de Melo > > ---------------------------------------------------------------- > Andi Kleen (1): > perf stat: Add extra output of counter values with -vv > > Arnaldo Carvalho de Melo (4): > perf trace: Do not print raw args list for syscalls with no args > perf trace: Move signum beautifier to tools/perf/trace/beauty/ > perf trace: Move open_flags beautifier to tools/perf/trace/beauty/ > perf trace: Move futex_op beautifier to tools/perf/trace/beauty/ > > Chris Phlipot (6): > perf callchain: Fix incorrect ordering of entries > perf tools: Refactor code to move call path handling out of thread-stack > perf script: Enable db export to output sampled callchains > perf script: Add call path id to exported sample in db export > perf script: Expose usage of the callchain db export via the python api > perf script: Update export-to-postgresql to support callchain export > > tools/perf/builtin-stat.c | 8 + > tools/perf/builtin-trace.c | 165 ++------------------- > tools/perf/scripts/python/export-to-postgresql.py | 47 +++--- > tools/perf/trace/beauty/futex_op.c | 44 ++++++ > tools/perf/trace/beauty/open_flags.c | 56 +++++++ > tools/perf/trace/beauty/signum.c | 53 +++++++ > tools/perf/util/Build | 1 + > tools/perf/util/call-path.c | 122 +++++++++++++++ > tools/perf/util/call-path.h | 77 ++++++++++ > tools/perf/util/db-export.c | 85 +++++++++++ > tools/perf/util/db-export.h | 3 + > tools/perf/util/machine.c | 56 +++++-- > .../util/scripting-engines/trace-event-python.c | 36 ++++- > tools/perf/util/thread-stack.c | 139 +---------------- > tools/perf/util/thread-stack.h | 31 ++-- > 15 files changed, 575 insertions(+), 348 deletions(-) > create mode 100644 tools/perf/trace/beauty/futex_op.c > create mode 100644 tools/perf/trace/beauty/open_flags.c > create mode 100644 tools/perf/trace/beauty/signum.c > create mode 100644 tools/perf/util/call-path.c > create mode 100644 tools/perf/util/call-path.h Pulled, thanks a lot Arnaldo! Ingo