Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751332AbbFDFtJ (ORCPT ); Thu, 4 Jun 2015 01:49:09 -0400 Received: from mail-wi0-f176.google.com ([209.85.212.176]:36427 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750769AbbFDFtA (ORCPT ); Thu, 4 Jun 2015 01:49:00 -0400 Date: Thu, 4 Jun 2015 07:48:54 +0200 From: Ingo Molnar To: Arnaldo Carvalho de Melo Cc: linux-kernel@vger.kernel.org, Adrian Hunter , Alexei Starovoitov , Brendan Gregg , Daniel Borkmann , David Ahern , He Kuang , Jiri Olsa , Kaixu Xia , Madhavan Srinivasan , Masami Hiramatsu , Michael Ellerman , Namhyung Kim , Peter Zijlstra , pi3orama@163.com, Sukadev Bhattiprolu , Wang Nan , Zefan Li , Arnaldo Carvalho de Melo Subject: Re: [GIT PULL 0/6] perf/core improvements and fixes Message-ID: <20150604054854.GA10969@gmail.com> References: <1433371238-14572-1-git-send-email-acme@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1433371238-14572-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 Content-Length: 4448 Lines: 113 * Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider applying. > > One of the next requests probably will have the eBPF work by Wang Nan, > but I am still going thru it and want to test it thoroughly. > > BTW: Have you looked at it lately? It is at: > > http://lkml.kernel.org/r/1433144296-74992-1-git-send-email-wangnan0@huawei.com > > Super summary from the above cover letter: > > --------------------- > It enables 'perf record' to filter events using eBPF programs like: > > # perf record --event bpf-file.o sleep 1 > > Events are selected and filtered according to definitions in bpf-file.o. Looks useful, but I think the UI needs one more tweak: could you fix it to be able to filter based on the eBPF _source_ file, not just the object file? People want to tweak such filters as they profile, so we should use the eBPF source code as the primary interface. We can compile it internally to the .o just fine. The .o file is a totally uninteresting intermediate product in itself. I.e. we need to first think through such profiling workflows from beginning to end before allowing them upstream. > --------------------- > > The first two patches from that series are in this pull req, as > they just move stuff into tools/include/linux/ from tools/perf/include. > > Regards, > > - Arnaldo > > The following changes since commit 5c9b9bc67c684e40b3a5e7e9facde0fb7200cd8c: > > Merge tag 'perf-core-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core (2015-05-29 20:19:02 +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 > > for you to fetch changes up to 1f121b03d058dd07199d8924373d3c52a207f63b: > > perf tools: Deal with kernel module names in '[]' correctly (2015-06-03 10:02:38 -0300) > > ---------------------------------------------------------------- > perf/core improvements and fixes: > > User visible: > > - Fix 'perf probe' segfault when glob matching function without debuginfo (Wang Nan) > > - Remove newline char when reading event scale and unit (Madhavan Srinivasan) > > - Deal with kernel module names in '[]' correctly (Wang Nan) > > Infrastructure: > > - Fix the search for the kernel DSO on the unified list (Arnaldo Carvalho de Melo) > > - Move tools/perf/util/include/linux/{kernel.h,list.h,poison.h} to tools/include, > to be used in tools/lib/bpf/ (Wang Nan) > > Signed-off-by: Arnaldo Carvalho de Melo > > ---------------------------------------------------------------- > Arnaldo Carvalho de Melo (1): > perf machine: Fix the search for the kernel DSO on the unified list > > Madhavan Srinivasan (1): > perf tools: Remove newline char when reading event scale and unit > > Wang Nan (4): > perf probe: Fix segfault when glob matching function without debuginfo > perf tools: Move linux/kernel.h to tools/include > tools: Move tools/perf/util/include/linux/{list.h,poison.h} to tools/include > perf tools: Deal with kernel module names in '[]' correctly > > tools/{perf/util => }/include/linux/kernel.h | 4 +- > tools/{perf/util => }/include/linux/list.h | 6 +-- > tools/include/linux/poison.h | 1 + > tools/perf/MANIFEST | 3 ++ > tools/perf/tests/kmod-path.c | 72 ++++++++++++++++++++++++++++ > tools/perf/util/dso.c | 47 ++++++++++++++++-- > tools/perf/util/dso.h | 2 +- > tools/perf/util/header.c | 8 ++-- > tools/perf/util/include/linux/poison.h | 1 - > tools/perf/util/machine.c | 22 ++++++++- > tools/perf/util/pmu.c | 11 ++++- > tools/perf/util/probe-event.c | 26 ++++++++-- > 12 files changed, 179 insertions(+), 24 deletions(-) > rename tools/{perf/util => }/include/linux/kernel.h (97%) > rename tools/{perf/util => }/include/linux/list.h (90%) > create mode 100644 tools/include/linux/poison.h > delete mode 100644 tools/perf/util/include/linux/poison.h Pulled, thanks a lot Arnaldo! Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/