Re: [PATCH] PM_RUN_CYC_SMT2_MODE event not suported on P9

[Ravi Bangoria]

Hi Mamatha,

On 10/26/2017 05:10 PM, Mamatha Inamdar wrote:
> As per the p9 event spreadsheet pm_run_cyc_smt2_mode is not listed.
> So this patch is to remove pm_run_cyc_smt2_mode from JSON file
>
> Signed-off-by: Mamatha Inamdar <mamatha4@xxxxxxxxxxxxxxxxxx>
> ---
>   .../perf/pmu-events/arch/powerpc/power9/cache.json |    7 +------
>   1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/tools/perf/pmu-events/arch/powerpc/power9/cache.json b/tools/perf/pmu-events/arch/powerpc/power9/cache.json
> index 18f6645..aecca97 100644
> --- a/tools/perf/pmu-events/arch/powerpc/power9/cache.json
> +++ b/tools/perf/pmu-events/arch/powerpc/power9/cache.json
> @@ -125,13 +125,8 @@
>       "BriefDescription": "Finish stall because the NTF instruction was a larx waiting to be satisfied"
>     },
>     {,
> -    "EventCode": "0x3006C",
> -    "EventName": "PM_RUN_CYC_SMT2_MODE",
> -    "BriefDescription": "Cycles in which this thread's run latch is set and the core is in SMT2 mode"
> -  },
> -  {,
>       "EventCode": "0x1C058",
>       "EventName": "PM_DTLB_MISS_16G",
>       "BriefDescription": "Data TLB Miss page size 16G"
>     }
> -]
> \ No newline at end of file
> +]

^^^ This 'newline thing' is unnecessary change. Can you please
resend the patch without it.

Otherwise, Ack-by: Ravi Bangoria <ravi.bangoria@xxxxxxxxxxxxxxxxxx>

Thanks,
Ravi