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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160521111703.GA5761@krava>
Date:	Sat, 21 May 2016 13:17:03 +0200
From:	Jiri Olsa <jolsa@...hat.com>
To:	Andi Kleen <andi@...stfloor.org>
Cc:	acme@...nel.org, jolsa@...nel.org, linux-kernel@...r.kernel.org,
	Andi Kleen <ak@...ux.intel.com>
Subject: Re: [PATCH 1/2] perf stat: Print topology/time headers with
 --metric-only

On Fri, May 20, 2016 at 12:50:14PM -0700, Andi Kleen wrote:
> From: Andi Kleen <ak@...ux.intel.com>
> 
> When --metric-only is enabled there were no headers for the topology
> in interval mode.  Also when headers were printed they were
> on a separate line.
> 
> Before:
> 
> $ perf stat  --metric-only  -A -I 1000 -a
>      1.001038376       frontend cycles idle insn per cycle       stalled cycles per insn branch-misses of all branches
>      1.001038376 CPU0     123.54%               0.23                5.29                    7.61%
>      1.001038376 CPU1     137.78%               0.24                5.13                   10.07%
>      1.001038376 CPU2      64.48%               0.22                5.50                    6.84%
> 
> After:
> 
> $ perf stat  --metric-only  -A -I 1000 -a
>      1.001111114 CPU0      82.46%               0.32                2.60                    7.64%
>      1.001111114 CPU1     126.63%               0.02               42.83                    0.15%
>      1.001111114 CPU2     193.54%               0.32                2.59                    6.92%
> 
> v2: Move all headers on a single line

changelog is wrong

[jolsa@...-x3650m4-01 perf]$ sudo ./perf stat  --metric-only  -A -I 1000 -a
#           time CPU frontend cycles idle backend cycles idle  insn per cycle       stalled cycles per insn branch-misses of all branches 
     1.003232468 CPU0      69.13%             56.43%                0.31                2.20                    0.01%                      
     1.003232468 CPU1      69.13%             56.23%                0.31                2.20                    0.00%                      
     1.003232468 CPU2      69.11%             56.27%                0.32                2.19                    0.00%                      

otherwise

Acked-by: Jiri Olsa <jolsa@...nel.org>

thanks,
jirka