Re: [perfmon2] [ RFC ] Enable collection of Nest PMUs on POWERPC platform

2015-07-09 Thread Stephane Eranian
Hi, Thanks for your patch. I have a few questions below. On Wed, Jul 8, 2015 at 2:00 AM, Deepthi Dharwar wrote: > > Enable collection of Nest PMUs on POWERPC platform > > From: Deepthi Dharwar > > Currently, POWER8 platforms supports reading of Nest/off-core > PMUs like memory read/write count

Re: [perfmon2] [ RFC ] Enable collection of Nest PMUs on POWERPC platform

2015-07-09 Thread Deepthi Dharwar
On 07/09/2015 02:44 PM, Stephane Eranian wrote: > Hi, > > Thanks for your patch. I have a few questions below. Hi, Thank you for reviewing the patch. > > On Wed, Jul 8, 2015 at 2:00 AM, Deepthi Dharwar > wrote: >> >> Enable collection of Nest PMUs on POWERPC platform >> >> From: Deepthi Dharw

Re: [perfmon2] [ RFC ] Enable collection of Nest PMUs on POWERPC platform

2015-07-09 Thread Stephane Eranian
Hi, Patch was applied. I would appreciate if you could submit another patch to add a couple of tests for the two events in tests/validate_power.c Thanks. On Thu, Jul 9, 2015 at 12:34 PM, Deepthi Dharwar wrote: > On 07/09/2015 02:44 PM, Stephane Eranian wrote: >> Hi, >> >> Thanks for your patc

Re: [perfmon2] [ RFC ] Enable collection of Nest PMUs on POWERPC platform

2015-07-09 Thread Deepthi Dharwar
On 07/10/2015 08:44 AM, Stephane Eranian wrote: > Hi, > > Patch was applied. Thank you so much. > > I would appreciate if you could submit another patch to add a couple of tests > for the two events in tests/validate_power.c > Sure. I will work this and send across the patch. Regards, Deepthi