linux-kernel - Re: [PATCH 0/5] perf tools: Store CPU cache details under perf data

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]

Date:	Tue, 16 Feb 2016 09:14:13 +0100
From:	Ingo Molnar <mingo@...nel.org>
To:	Jiri Olsa <jolsa@...nel.org>
Cc:	Arnaldo Carvalho de Melo <acme@...nel.org>,
	lkml <linux-kernel@...r.kernel.org>,
	David Ahern <dsahern@...il.com>,
	Namhyung Kim <namhyung@...nel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>
Subject: Re: [PATCH 0/5] perf tools: Store CPU cache details under perf data


* Jiri Olsa <jolsa@...nel.org> wrote:

> hi,
> adding support to store CPU cache details under perf data. 
> 
>   $ perf report --header-only -I
>   ...
>   # cache info:
>   #  L1 Data 32K [0-1]
>   #  L1 Instruction 32K [0-1]
>   #  L1 Data 32K [2-3]
>   #  L1 Instruction 32K [2-3]
>   #  L2 Unified 256K [0-1]
>   #  L2 Unified 256K [2-3]
>   #  L3 Unified 4096K [0-3]

Very small UI nit, wouldn't it be nicer if this displayed tabular output, and if 
numbers were adjusted to their decimal point, i.e. something like:

>   # CPU cache info:
>   #  L1 Data			  32K	[0-1]
>   #  L1 Instruction		  32K	[0-1]
>   #  L1 Data			  32K	[2-3]
>   #  L1 Instruction		  32K	[2-3]
>   #  L2 Unified		 256K	[0-1]
>   #  L2 Unified		 256K	[2-3]
>   #  L3 Unified		4096K	[0-3]

(Also note that it says 'CPU cache info' - there are a lot of caches in the 
system.)

Thanks,

	Ingo