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

[Ingo Molnar <mingo@kernel.org>]

* Jiri Olsa <jolsa@kernel.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