From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752625AbcF2CfM (ORCPT ); Tue, 28 Jun 2016 22:35:12 -0400 Received: from LGEAMRELO11.lge.com ([156.147.23.51]:54976 "EHLO lgeamrelo11.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752449AbcF2CfK convert rfc822-to-8bit (ORCPT ); Tue, 28 Jun 2016 22:35:10 -0400 X-Original-SENDERIP: 156.147.1.125 X-Original-MAILFROM: namhyung@kernel.org X-Original-SENDERIP: 165.244.98.203 X-Original-MAILFROM: namhyung@kernel.org X-Original-SENDERIP: 10.177.227.17 X-Original-MAILFROM: namhyung@kernel.org Date: Wed, 29 Jun 2016 11:20:06 +0900 From: Namhyung Kim To: David Ahern CC: Arnaldo Carvalho de Melo , Jiri Olsa , lkml Subject: Re: [BUG] perf test 29 crashing Message-ID: <20160629022006.GC1628@sejong> References: <20160628123410.GA1862@krava> <20160628165914.GJ27027@kernel.org> <6939ccab-dc51-27ab-69c0-9600f3a7ed6f@gmail.com> MIME-Version: 1.0 In-Reply-To: <6939ccab-dc51-27ab-69c0-9600f3a7ed6f@gmail.com> User-Agent: Mutt/1.6.1 (2016-04-27) X-MIMETrack: Itemize by SMTP Server on LGEKRMHUB07/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2016/06/29 11:20:06, Serialize by Router on LGEKRMHUB07/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2016/06/29 11:20:06 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Tue, Jun 28, 2016 at 11:52:09AM -0600, David Ahern wrote: > On 6/28/16 10:59 AM, Arnaldo Carvalho de Melo wrote: > > Em Tue, Jun 28, 2016 at 02:34:10PM +0200, Jiri Olsa escreveu: > > > hi, > > > I'm having test 29 crashing on latest Arnaldo's perf/core: > > > > > > [jolsa@krava perf]$ ./perf test hist > > > 15: Test matching and linking multiple hists : Ok > > > 25: Test filtering hist entries : Ok > > > 28: Test output sorting of hist entries : Ok > > > 29: Test cumulation of child hist entries : FAILED! > > > > > > I was hunting this, because I though it was caused by one > > > of my local changes, but then realized it's public bug ;-) > > > > > > but can't find the reason so far.. > > > > Huh? > > > > [acme@jouet linux]$ perf test 29 > > 29: Test cumulation of child hist entries : Ok > > [acme@jouet linux]$ perf --version > > perf version 4.7.rc5.g1c1e35c > > [acme@jouet linux]$ > > > > If this helps break the tie: > > dsa@kenny:~/kernel.git$ /tmp/perf/perf test 29 > 29: Test cumulation of child hist entries : Ok > > dsa@kenny:~/kernel.git$ /tmp/perf/perf --version > perf version 4.7.rc5.g02184c > > But running on net-next kernel from a few days ago + 1 local patch: > dsa@kenny:~/kernel.git$ uname -r > 4.7.0-rc2-00862-g23854048c919 > > (I do get a few other failures) Running latest acme/perf/core: $ perf test hist 15: Test matching and linking multiple hists : Ok 25: Test filtering hist entries : Ok 28: Test output sorting of hist entries : Ok 29: Test cumulation of child hist entries : Ok $ perf --version perf version 4.6.g6d9c67 But I saw a build failure which was fixed by below patch: >>From 0b011dbf5ba60f78dd3198dc3f72005334507eb4 Mon Sep 17 00:00:00 2001 From: Namhyung Kim Date: Wed, 29 Jun 2016 11:10:22 +0900 Subject: [PATCH] perf tools: Fix build failture on perl script context MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CC scripts/perl/Perf-Trace-Util/Context.o In file included from /usr/lib/perl5/core_perl/CORE/perl.h:3905:0, from Context.xs:23: /usr/lib/perl5/core_perl/CORE/inline.h: In function ‘S_cx_popsub_args’: /usr/lib/perl5/core_perl/CORE/cop.h:612:13: warning: declaration of ‘av' shadows a previous local [-Werror=shadow] AV *av = GvAV(PL_defgv); ^ /usr/lib/perl5/core_perl/CORE/inline.h:526:5: note: in expansion of macro ‘CX_POP_SAVEARRAY’ CX_POP_SAVEARRAY(cx); ^~~~~~~~~~~~~~~~ In file included from /usr/lib/perl5/core_perl/CORE/perl.h:5853:0, from Context.xs:23: /usr/lib/perl5/core_perl/CORE/inline.h:518:9: note: shadowed declaration is here AV *av; ^~ Signed-off-by: Namhyung Kim --- tools/perf/scripts/perl/Perf-Trace-Util/Build | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/perf/scripts/perl/Perf-Trace-Util/Build b/tools/perf/scripts/perl/Perf-Trace-Util/Build index 928e110179cb..34faecf774ae 100644 --- a/tools/perf/scripts/perl/Perf-Trace-Util/Build +++ b/tools/perf/scripts/perl/Perf-Trace-Util/Build @@ -1,3 +1,5 @@ libperf-y += Context.o -CFLAGS_Context.o += $(PERL_EMBED_CCOPTS) -Wno-redundant-decls -Wno-strict-prototypes -Wno-unused-parameter -Wno-nested-externs -Wno-undef -Wno-switch-default +CFLAGS_Context.o += $(PERL_EMBED_CCOPTS) -Wno-redundant-decls -Wno-strict-prototypes +CFLAGS_Context.o += -Wno-unused-parameter -Wno-nested-externs -Wno-undef +CFLAGS_Context.o += -Wno-switch-default -Wno-shadow -- 2.9.0