Re: [PATCHv9 0/3] perf stat: Add scripting support

[Jiri Olsa]

On Wed, Jan 06, 2016 at 12:05:41PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Jan 06, 2016 at 03:38:53PM +0100, Jiri Olsa escreveu:
> > On Wed, Jan 06, 2016 at 11:18:02AM -0300, Arnaldo Carvalho de Melo wrote:
> > > Em Wed, Jan 06, 2016 at 11:49:54AM +0100, Jiri Olsa escreveu:
> > > > hi,
> > > > sending another version of stat scripting.
> > >  
> > > > v9 changes:
> > > >   - rebased on top of accepted patches
> > > >   - desribed CPI metric in changelog [Arnaldo]
> > > >   - fixed cpu conversion [Arnaldo]
> > > 
> > > Thanks, applied. Testing the cpi script with an endless:
> > > 
> > > perf stat record -I 1000 -a | perf script -s ~acme/git/linux/tools/perf/scripts/python/stat-cpi.py
> > >      187.151796: cpu -1, thread -1 -> cpi 0.797917 (2568467461/3218963700)
> > >      188.151979: cpu -1, thread -1 -> cpi 0.734628 (2714373371/3694892981)
> > >      189.152212: cpu -1, thread -1 -> cpi 0.753332 (2958819204/3927644236)
> > >      190.152975: cpu -1, thread -1 -> cpi 1.587754 (202360895/127451009)
> > >      191.153486: cpu -1, thread -1 -> cpi 1.558219 (290557309/186467579)
> > > 
> > > in one monitor while trying various workloads in another, to see how it reacts,
> > > and after a while those raw numbers at the end of each line become just noise,
> > > I think they should be shown only under 'stat-cpy -v' (I think we can pass args
> > > to the 'perf script' scripts, right?
> > > 
> > > Also the 'cpu', 'thread' and '-> cpi' could be turned into headers, like 'perf
> > > stat' does?
> > > 
> > > Anyway, you called it 'example script' and we can improve it on top of what I
> > > pushed to perf/core, i.e. this patchkit, unchanged, thanks!
> > 
> > yep, it's just noise and it's just an example to show
> 
> Hey, its not "just noise", I meant just the "(2568467461/3218963700)"
> part at the end of each line and the repetitive cpu/thread/cpi stuff
> that can be moved to the first line, as headers.

yep, thats what I meant as well

jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/