Re: [PATCH] perf stat: Print topology/time headers with --metric-only

[Andi Kleen]

On Fri, May 20, 2016 at 05:47:38PM +0200, Jiri Olsa wrote:
> On Fri, May 20, 2016 at 08:43:19AM -0700, Andi Kleen wrote:
> > > hum, I'm getting something else:
> > > 
> > > [jolsa@ibm-x3650m4-01 perf]$ sudo ./perf stat --topdown -I 1000 -a
> > > nmi_watchdog enabled with topdown. May give wrong results.
> > > Disable with echo 0 > /proc/sys/kernel/nmi_watchdog
> > > #           time core         cpus
> > >      1.001710838                   retiring             bad speculation      frontend bound       backend bound        
> > >      1.001710838 S0-C0           2     38.1%               -0.0%               58.8%                3.1%           
> > >      1.001710838 S0-C1           2     38.0%                0.0%               59.4%                2.5%           
> > > 
> > > 
> > > [jolsa@ibm-x3650m4-01 perf]$ sudo ./perf stat --metric-only -e cycles,instructions -a -I 1000
> > > #           time
> > >      1.000756338 insn per cycle       stalled cycles per insn 
> > >      1.000756338    0.32                                      
> > >      2.001155562    0.30                                      
> > 
> > Right sorry, that was me not cutting and pasting correctly.
> > 
> > Your output is correct and intended. 
> 
> hum, I dont think so.. headers should be on the same line, right?

I posted new patches which move it to the same line, and also
fix another bug I noticed.

-Andi

-- 
ak@xxxxxxxxxxxxxxx -- Speaking for myself only