From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752385AbcF3T5F (ORCPT ); Thu, 30 Jun 2016 15:57:05 -0400 Received: from mail.kernel.org ([198.145.29.136]:36154 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751460AbcF3T5D (ORCPT ); Thu, 30 Jun 2016 15:57:03 -0400 Date: Thu, 30 Jun 2016 16:56:56 -0300 From: Arnaldo Carvalho de Melo To: Jiri Olsa Cc: Namhyung Kim , David Ahern , Arnaldo Carvalho de Melo , lkml Subject: Re: [BUG] perf test 29 crashing Message-ID: <20160630195656.GD5324@kernel.org> References: <20160628123410.GA1862@krava> <20160628165914.GJ27027@kernel.org> <6939ccab-dc51-27ab-69c0-9600f3a7ed6f@gmail.com> <20160629022006.GC1628@sejong> <20160630151016.GA4828@krava> <20160630161816.GA2939@krava> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160630161816.GA2939@krava> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.6.1 (2016-04-27) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Thu, Jun 30, 2016 at 06:18:16PM +0200, Jiri Olsa escreveu: > On Thu, Jun 30, 2016 at 05:10:16PM +0200, Jiri Olsa wrote: > > SNIP > > > > > hate to be the only one with this.. now I need to dive in ;-) > > it was the setup in my .perfconfig: > > [call-graph] > threshold=10 > > caused some of the callchains to disappear and screw the test, > I think we should make that test using default values, like in > attached patch > > however maybe we should think about disabling ~/.perfconfig > overload for perf test at all > > I'll post patch below with my other stuff if there are no > objection or another solution Yeah, tests expect things in a certain way, so shouldn't be affect by local .perfconfig stuff, ignoring it in tests is the way to go. But probably we should instead check if the subcommand is 'test' when calling into perf_config(), no? I.e. Other tests may suffer from this issue. - Arnaldo > thanks, > jirka > > --- > diff --git a/tools/perf/tests/hists_cumulate.c b/tools/perf/tests/hists_cumulate.c > index a9e3db3afac4..1208284a7338 100644 > --- a/tools/perf/tests/hists_cumulate.c > +++ b/tools/perf/tests/hists_cumulate.c > @@ -666,6 +666,8 @@ static int test4(struct perf_evsel *evsel, struct machine *machine) > perf_evsel__set_sample_bit(evsel, CALLCHAIN); > > setup_sorting(NULL); > + > + callchain_param = callchain_param_default; > callchain_register_param(&callchain_param); > > err = add_hist_entries(hists, machine); > diff --git a/tools/perf/util/callchain.h b/tools/perf/util/callchain.h > index a70f6b54eb92..13e75549c440 100644 > --- a/tools/perf/util/callchain.h > +++ b/tools/perf/util/callchain.h > @@ -106,6 +106,7 @@ struct callchain_param { > }; > > extern struct callchain_param callchain_param; > +extern struct callchain_param callchain_param_default; > > struct callchain_list { > u64 ip; > diff --git a/tools/perf/util/util.c b/tools/perf/util/util.c > index e08b9a092a23..5f44a21955cd 100644 > --- a/tools/perf/util/util.c > +++ b/tools/perf/util/util.c > @@ -19,12 +19,19 @@ > #include "callchain.h" > #include "strlist.h" > > -struct callchain_param callchain_param = { > - .mode = CHAIN_GRAPH_ABS, > - .min_percent = 0.5, > - .order = ORDER_CALLEE, > - .key = CCKEY_FUNCTION, > - .value = CCVAL_PERCENT, > +#define CALLCHAIN_PARAM_DEFAULT \ > + .mode = CHAIN_GRAPH_ABS, \ > + .min_percent = 0.5, \ > + .order = ORDER_CALLEE, \ > + .key = CCKEY_FUNCTION, \ > + .value = CCVAL_PERCENT, \ > + > +struct callchain_param callchain_param = { > + CALLCHAIN_PARAM_DEFAULT > +}; > + > +struct callchain_param callchain_param_default = { > + CALLCHAIN_PARAM_DEFAULT > }; > > /*