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

From: Andi Kleen
Date: Fri May 20 2016 - 15:53:23 EST


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