Re: [BUG] perf test 29 crashing

[Namhyung Kim <namhyung@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 <namhyung@kernel.org>
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 <namhyung@kernel.org>
---
 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