2017-11-16 07:58:51

by Ravi Bangoria

[permalink] [raw]
Subject: Re: [PATCH] PM_RUN_CYC_SMT2_MODE event not suported on P9

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 <[email protected]>
> ---
> .../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 <[email protected]>

Thanks,
Ravi


From 1582320188662182016@xxx Thu Oct 26 11:42:04 +0000 2017
X-GM-THRID: 1582320188662182016
X-Gmail-Labels: Inbox,Category Forums