lkml.org 
[lkml]   [2019]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: perf: python3 + tests attr.py failing
On Tue, Feb 05, 2019 at 10:49:35AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Feb 04, 2019 at 09:37:38PM +0100, Jiri Olsa escreveu:
> > On Mon, Feb 04, 2019 at 12:10:42PM -0300, Arnaldo Carvalho de Melo wrote:
> > > Noticed this now when building with PYTHON=python3, can you take a look?
> > > For now I'm reinstalling python2-devel and building with it, i.e.
> > > removing that PYTHON=python3 make command line variable.
> > >
> > > [root@quaco ~]# perf test -v perf_event_attr
> > > 16: Setup struct perf_event_attr :
> > > --- start ---
> > > test child forked, pid 23937
> > > File "/home/acme/libexec/perf-core/tests/attr.py", line 325
> > > except Unsup, obj:
> > > ^
> > > SyntaxError: invalid syntax
> > > test child finished with -1
> > > ---- end ----
> > > Setup struct perf_event_attr: FAILED!
> >
> > hum, I built the same way like you 'make PYTHON=python3'
> > [root@krava perf]# ldd ./perf | grep python
> > libpython3.6m.so.1.0 => /lib64/libpython3.6m.so.1.0 (0x00007f4ca7cc4000)
> >
> > and get 'Ok result'
> >
> > [root@krava perf]# ./perf test attr
> > 16: Setup struct perf_event_attr : Ok
> > 48: Synthesize attr update : Ok
> >
> >
> > [root@krava perf]# python3 --version
> > Python 3.6.8
> > [root@krava perf]# rpm -qa | grep python3-devel
> > python3-devel-3.6.8-1.fc28.x86_64
> >
> > which Fedora are you on 29? I'm on 28
>
> I think you tested perf/core, that has Tony's fix to make 'perf test perf_event_attr'
> work with both python2 and python3, right?
>
> While I was having that failure with perf/urgent.
>
> I just cherry-picked:
>
> commit 35ea7e4bbb89ecd32057f5f6a2a8feb0d7224e51
> Author: Tony Jones <tonyj@suse.de>
> Date: Wed Jan 23 16:52:29 2019 -0800
>
> perf script python: Add Python3 support to tests/attr.py

right, i was over latest perf/core

jirka

>
>
> ----------------------
>
> And now my perf/urgent branch passes that 'perf test' entry with both
> python2 and 3.
>
> - Arnaldo

\
 
 \ /
  Last update: 2019-02-05 16:43    [W:0.089 / U:4.244 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site