Messages in this thread | | | Date | Thu, 24 Jan 2013 17:58:13 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 0/4] perf tools: some 'perf script' fixes |
| |
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 >
| |