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

[Jiri Olsa]

On Tue, Feb 16, 2016 at 09:14:13AM +0100, Ingo Molnar wrote:
> 
> * Jiri Olsa <jolsa@xxxxxxxxxx> 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.)

yep, looks better.. will change

thanks,
jirka