Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] PM_RUN_CYC_SMT2_MODE event not suported on P9
To:     Mamatha Inamdar <mamatha4@linux.vnet.ibm.com>
References: <150901800962.22452.14521683793785014245.stgit@localhost.localdomain>
Cc:     linux-kernel@vger.kernel.org, sukadev@linux.vnet.ibm.com,
        ravi.bangoria@linux.vnet.ibm.com
From:   Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>
Date:   Thu, 16 Nov 2017 10:17:43 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <150901800962.22452.14521683793785014245.stgit@localhost.localdomain>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Message-Id: <addb2ff0-7d0d-75f5-5206-89d7b0dcf11a@linux.vnet.ibm.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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@linux.vnet.ibm.com>
> ---
>   .../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@linux.vnet.ibm.com>

Thanks,
Ravi


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