From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754956AbcBBOQp (ORCPT ); Tue, 2 Feb 2016 09:16:45 -0500 Received: from mail-pf0-f196.google.com ([209.85.192.196]:33566 "EHLO mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754914AbcBBOQn (ORCPT ); Tue, 2 Feb 2016 09:16:43 -0500 Date: Tue, 2 Feb 2016 23:15:35 +0900 From: Namhyung Kim To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Peter Zijlstra , Jiri Olsa , LKML , Andi Kleen , David Ahern , Frederic Weisbecker , Wang Nan Subject: Re: [PATCH 03/10] perf report: Apply --percent-limit to callchains also Message-ID: <20160202141535.GA4627@danjae.kornet> References: <1453909257-26015-1-git-send-email-namhyung@kernel.org> <1453909257-26015-4-git-send-email-namhyung@kernel.org> <20160201201936.GC20817@kernel.org> <20160202130537.GA4153@danjae.kornet> <20160202135534.GB29970@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160202135534.GB29970@kernel.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 02, 2016 at 10:55:34AM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 02, 2016 at 10:05:37PM +0900, Namhyung Kim escreveu: > > Hi Arnaldo, > > > > On Mon, Feb 01, 2016 at 05:19:36PM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Thu, Jan 28, 2016 at 12:40:50AM +0900, Namhyung Kim escreveu: > > > > Currently --percent-limit option only works for hist entries. However > > > > it'd be better to have same effect to callchains as well > > > > > > Documentation needs updating? It says: > > > > > > --percent-limit:: > > > Do not show entries which have an overhead under that percent. > > > (Default: 0). > > > > Right. Is it ok to you? > > > > --percent-limit:: > > Do not show entries and callchains which have an overhead under that > > percent. (Default: 0). > > Ok, but is the default zero? > > That was what I was alluding to, as as soon as I applied the patch that > made callchains honour this limit, about 60% of the entries in the > particular perf.data file I was 'perf report'ing lost its '+' (callchain > expansion) signs. Yes, and this is what I want to say too. :) The default value of percent limit is different for hist entry and callchains. For hist entry the default is 0, and for callchains it's 0.5%. But using --percent-limit option, we can set both at once from now on. Before this patchset, percent limit of callchains didn't checked. Once applied, users can see callchains are disppeared like your case. This is just because 0.5% of the default limit for callchains is too high IMHO - I guess it was originally set for 'fractal' mode which calculates relative percents. So I proposed to change the default in the patch 10/10 but Andi thoughts 0.005% was too small. We need to choose other value like 0.05% ? Thanks, Namhyung