From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755132AbdBGQDL (ORCPT ); Tue, 7 Feb 2017 11:03:11 -0500 Received: from mail.kernel.org ([198.145.29.136]:35938 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754500AbdBGQCa (ORCPT ); Tue, 7 Feb 2017 11:02:30 -0500 Date: Tue, 7 Feb 2017 13:02:14 -0300 From: Arnaldo Carvalho de Melo To: Namhyung Kim Cc: Paul McKenney , Ingo Molnar , Peter Zijlstra , Jiri Olsa , LKML , Minchan Kim , Taeung Song Subject: Re: [PATCHSET 0/3] perf diff: Introduce delta-abs compute method Message-ID: <20170207160214.GK24988@kernel.org> References: <20170206072037.8189-1-namhyung@kernel.org> <20170206125149.GF3762@kernel.org> <20170206142616.GC2156@danjae.aot.lge.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170206142616.GC2156@danjae.aot.lge.com> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Mon, Feb 06, 2017 at 11:26:16PM +0900, Namhyung Kim escreveu: > Hi Arnaldo, > > On Mon, Feb 06, 2017 at 09:51:49AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Mon, Feb 06, 2017 at 04:20:34PM +0900, Namhyung Kim escreveu: > > I agree on having the default changed to 'delta-abs', Ingo? > Good. Also, as I said in the changelog, it needs to change default > value of -o option to 1 in order to make the 'delta-abs' effective. ok > > Namhyung, and perhaps we should have a single letter option to do that > > '| grep -v ^#' bit :-) and perhaps we also should have, for all tools > > the equivalent of that "| head", that git log has: > > > > [acme@jouet linux]$ git log --oneline -5 > > d7cb3a507d23 Merge tag 'perf-core-for-mingo-4.11-20170201' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core > > 5443624bedd0 perf/x86/intel/pt: Add format strings for PTWRITE and power event tracing > > b05d1093987a perf ftrace: Add ftrace.tracer config option > > 43d41deb71fe perf tools: Create for_each_event macro for tracepoints iteration > > a26305363d4b perf test: Add libbpf pinning test > > [acme@jouet linux]$ > > > > That '-5' to show just the first 5 lines worth of output. > > > > With all that we would have: > > > > perf diff -o 1 -q10 > > > > As the equivalent to "perf diff -o 1 -c delta-abs | grep -v ^# | head". > > The -q/--quiet looks ok since it corresponds to -v/--verbose option. Ok, agreed on this one. > But I'm not sure about the number option. > In case of git, it'll stop processing commits after the given number > of them, so it will reduce significant processing time IMHO. However, > in perf, we need to process whole data anyway and sort at the final > stage, and then stop displaying entries after the given number. > Maybe it's just a shortcut of piping to the head command. Then I I wasn't thinking about the processing savings from stopping to process at that many lines, my suggestion was just about making the command line more compact, to type less. If that can also map to processing savings, the better. - Arnaldo > don't feel the strong desire to have it as we have pager, TUI and GUI > already. > > Thanks, > Namhyung > > > > > > Ah, adding Paul McKenney to the CC list, he may have something to add > > here. > > > > - Arnaldo > > > > > The code is avaiable at 'perf/diff-delta-abs-v1' branch in > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git > > > > > > Thanks, > > > Namhyung > > > > > > > > > Namhyung Kim (3): > > > perf diff: Add 'delta-abs' compute method > > > perf diff: Add diff.order config option > > > perf diff: Add diff.compute config option > > > > > > tools/perf/Documentation/perf-config.txt | 12 +++++ > > > tools/perf/Documentation/perf-diff.txt | 15 +++++-- > > > tools/perf/builtin-diff.c | 76 ++++++++++++++++++++++++++++++-- > > > 3 files changed, 97 insertions(+), 6 deletions(-) > > > > > > -- > > > 2.11.0