Re: [PATCH 0/4] perf tools: some 'perf script' fixes

[Arnaldo Carvalho de Melo <acme@redhat.com>]

Em Thu, Jan 24, 2013 at 02:33:53PM -0600, Tom Zanussi escreveu:
> On Thu, 2013-01-24 at 15:27 -0300, Arnaldo Carvalho de Melo wrote:
> > Em Fri, Jan 18, 2013 at 01:51:24PM -0600, Tom Zanussi escreveu:
> > > This patchset fixes various problems I hit while doing a run-through
> > > of the existing 'perf script' scripts.
> > > 
> > > Also, sometimes when running rwtop I get a 'failed to read event
> > > header' error, which this patch fixes:
> > > 
> > > https://lkml.org/lkml/2012/9/14/401
> > > 
> > > Not sure if that was meant to get picked up, but it fixed that
> > > particular problem here for me.
> > 
> > David, can you please resubmit it with an explanation in the commit
> > message?
> > 
> > Tom, can I add a Tested-by: you to such patch?
> > 
> 
> Yeah, sure.
> 
> Tested-by: Tom Zanussi <tom.zanussi@linux.intel.com>

David, perhaps you should just add a local readn variant for 'script'
till we find a more general solution, at least it fixes script's case.

- Arnaldo
 
> Thanks,
> 
> Tom
> 
> > > Tom Zanussi (4):
> > >   perf/tools: reinstate 'signed' field flag for tracepoints
> > >   perf script: don't display trace info when invoking scripts
> > >   perf tools: hook up perf_scripting_context->pevent
> > >   perf script: remove workqueue-stats script
> > 
> > Thanks, applied to my perf/core branch at:
> > 
> > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux
> > 
> > - Arnaldo
>