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

Reply via email to