Re: [PATCH] perf vendor events: Initial json/events list for power10 platform

From: kajoljain
Date: Mon Apr 19 2021 - 06:26:11 EST




On 4/19/21 2:38 AM, Paul A. Clarke wrote:
> On Sat, Apr 17, 2021 at 02:48:50PM +0530, Kajol Jain wrote:
>> Patch adds initial json/events for POWER10.
>
> I was able to apply, build, and run perf with these changes,
> and every new event at least ran successfully with
> `perf stat`.
>
> Pedantically, there is a lot of inconsistency as to whether
> the `BriefDescription` ends with a period or not, and whether
> there is an extra space at the end.

Hi Paul,
Thanks for reviewing the patch. Sure I will remove this inconsistency
and send v2 patch for the same, with your Tested-by and Reviewed-by tag.

Thanks,
Kajol Jain

>
> Regardless, LGTM.
>
> Tested-by: Paul A. Clarke <pc@xxxxxxxxxx>
> Reviewed-by: Paul A. Clarke <pc@xxxxxxxxxx>
>
> PC
>