Re: [PATCH v8 0/4] perf: arm-spe: Decode SPE source and use for perf c2c

Joe Mario
2022-05-23T17:26:03Z

On 5/22/22 2:15 AM, Leo Yan wrote:
> Hi Joe,
>
> On Thu, May 19, 2022 at 11:16:53AM -0400, Joe Mario wrote:
>
> [SNIP]
>
>> Last Comment:
>> There's a row in the Pareto table that has incorrect column alignment.
>> Look at row 80 below in the truncated snipit of output. It has an extra field inserted in it at the beginning.
>> I also show what the corrected output should look like.
>>
>> Incorrect row 80:
>> 71 =================================================
>> 72 Shared Cache Line Distribution Pareto
>> 73 =================================================
>> 74 #
>> 75 # ----- HITM ----- Snoop ------- Store Refs ------ ------- CL --------
>> 76 # RmtHitm LclHitm Peer L1 Hit L1 Miss N/A Off Node PA cnt Code address
>> 77 # ....... ....... ....... ....... ....... ....... ..... .... ...... ..................
>> 78 #
>> 79 -------------------------------------------------------------------------------
>> 80 0 0 0 4648 0 0 11572 0x422140
>> 81 -------------------------------------------------------------------------------
>> 82 0.00% 0.00% 0.00% 0.00% 0.00% 44.47% 0x0 N/A 0 0x400ce8
>> 83 0.00% 0.00% 10.26% 0.00% 0.00% 0.00% 0x0 N/A 0 0x400e48
>> 84 0.00% 0.00% 0.00% 0.00% 0.00% 55.53% 0x0 N/A 0 0x400e54
>> 85 0.00% 0.00% 89.74% 0.00% 0.00% 0.00% 0x8 N/A 0 0x401038
>>
>>
>> Corrected row 80:
>> 71 =================================================
>> 72 Shared Cache Line Distribution Pareto
>> 73 =================================================
>> 74 #
>> 75 # ----- HITM ----- Snoop ------- Store Refs ----- ------- CL --------
>> 76 # RmtHitm LclHitm Peer L1 Hit L1 Miss N/A Off Node PA cnt Code address
>> 77 # ....... ....... ....... ....... ....... ...... ..... .... ...... ..................
>> 78 #
>> 79 -------------------------------------------------------------------------------
>> 80 0 0 4648 0 0 11572 0x422140
>> 81 -------------------------------------------------------------------------------
>> 82 0.00% 0.00% 0.00% 0.00% 0.00% 44.47% 0x0 N/A 0 0x400ce8
>> 83 0.00% 0.00% 10.26% 0.00% 0.00% 0.00% 0x0 N/A 0 0x400e48
>> 84 0.00% 0.00% 0.00% 0.00% 0.00% 55.53% 0x0 N/A 0 0x400e54
>> 85 0.00% 0.00% 89.74% 0.00% 0.00% 0.00% 0x8 N/A 0 0x401038
>
> Hmm‥. At my side, I used below command to output pareto view, but I
> cannot see the conlumn "CL", the conlumn "CL" is only shown for TUI
> mode but not for the mode "--stdio". Could you share the method for
> how to reproduce this issue?

Hi Leo:
I figured out why my output was different than yours.

I did not have the slang-devel rpm installed on the host system.

In my original perf build, I missed the this output in the build log:
> slang not found, disables TUI support. Please install slang-devel, libslang-dev or libslang2-dev

Once I installed slang-devel, rebuilt perf, and then reran my test, the pareto output looked fine.

When the TUI support is disabled, it shouldn't corrupt the resulting stdio output. I don't believe this has anything to do with your commits.

Last, it looks like you should update the help text for the display flag options to reflect your new peer option.
Currently it says:
-d, --display <Switch HITM output type>
lcl,rmt

But since you added the "peer" display, shouldn't the output for that help text state:
-d, --display <Switch HITM output type>
lcl,rmt,peer

Joe