From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Olsa Subject: Re: [GIT PULL 0/6] perf/urgent fixes Date: Fri, 6 Dec 2019 16:04:55 +0100 Message-ID: <20191206150455.GC31721@krava> References: <20191205193224.24629-1-acme@kernel.org> <20191206075701.GA25384@gmail.com> <20191206142516.GA31721@krava> <20191206144354.GD30698@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20191206144354.GD30698@kernel.org> Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Thomas Gleixner , Jiri Olsa , Namhyung Kim , Clark Williams , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Adrian Hunter , Ravi Bangoria , Arnaldo Carvalho de Melo List-Id: linux-perf-users.vger.kernel.org On Fri, Dec 06, 2019 at 11:43:54AM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Dec 06, 2019 at 03:25:16PM +0100, Jiri Olsa escreveu: > > On Fri, Dec 06, 2019 at 08:57:01AM +0100, Ingo Molnar wrote: > >=20 > > SNIP > >=20 > > > > tools/include/uapi/drm/drm.h | 3 +- > > > > tools/include/uapi/drm/i915_drm.h | 128 ++++++++++++++++++++++++++= +++++++++++- > > > > tools/perf/builtin-inject.c | 13 +--- > > > > tools/perf/builtin-report.c | 8 +++ > > > > tools/perf/util/sort.c | 16 +++-- > > > > 5 files changed, 147 insertions(+), 21 deletions(-) > > >=20 > > > Pulled, thanks a lot Arnaldo! > > >=20 > > > JFYI, on my system the default perf/urgent build still has this noise= =20 > > > generated by util/parse-events.y and util/expr.y: > > >=20 > > > util/parse-events.y:1.1-12: warning: deprecated directive, use =E2= =80=98%define api.pure=E2=80=99 [-Wdeprecated] > > > 1 | %pure-parser > > > | ^~~~~~~~~~~~ > > > util/parse-events.y: warning: fix-its can be applied. Rerun with o= ption '--update'. [-Wother] > > > util/expr.y:15.1-12: warning: deprecated directive, use =E2=80=98%d= efine api.pure=E2=80=99 [-Wdeprecated] > > > 15 | %pure-parser > > > | ^~~~~~~~~~~~ > > > util/expr.y: warning: fix-its can be applied. Rerun with option '-= -update'. [-Wother] > >=20 > > just saw it in fedora 31 with new bison, change below > > should fix it, I'll post it with other fixes later >=20 > As I explained to Ingo, this will make it fail with older systems, for > now this is just a warning, thus I've not been eager to get this merged, > Andi alredy submitted this, for instance. >=20 > Is there some way to have some sort of ifdef based on bison's version so > that we can have both? I see, I guess we could use one or another based on bison version with macro jirka >=20 > At some point I'll just bite the bullet and stop testing on such older > systems, but while this is not strictly needed... >=20 > - Arnaldo > =20 > > jirka > >=20 > > --- > > diff --git a/tools/perf/util/expr.y b/tools/perf/util/expr.y > > index f9a20a39b64a..4ef801334b9d 100644 > > --- a/tools/perf/util/expr.y > > +++ b/tools/perf/util/expr.y > > @@ -12,7 +12,7 @@ > > #define MAXIDLEN 256 > > %} > > =20 > > -%pure-parser > > +%define api.pure > > %parse-param { double *final_val } > > %parse-param { struct parse_ctx *ctx } > > %parse-param { const char **pp } > > diff --git a/tools/perf/util/parse-events.y b/tools/perf/util/parse-eve= nts.y > > index e2eea4e601b4..87a0d11676f0 100644 > > --- a/tools/perf/util/parse-events.y > > +++ b/tools/perf/util/parse-events.y > > @@ -1,4 +1,4 @@ > > -%pure-parser > > +%define api.pure > > %parse-param {void *_parse_state} > > %parse-param {void *scanner} > > %lex-param {void* scanner} >=20 > --=20 >=20 > - Arnaldo >=20