Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 11:25 +0100, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: > > It's a perrenial request from our hardware PMU folks to be able to see the > > raw > > values of the PMU registers. > > > > I think partly it's so that they can

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 11:25 +0100, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: > > It's a perrenial request from our hardware PMU folks to be able to see the > > raw > > values of the PMU registers. > > > > I think partly it's so that they can

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 03:55 PM, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: >> It's a perrenial request from our hardware PMU folks to be able to see the >> raw >> values of the PMU registers. >> >> I think partly it's so that they can verify

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 03:34 PM, Michael Ellerman wrote: > On Fri, 2015-11-06 at 10:24 +0100, Peter Zijlstra wrote: >> On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: >>> On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: On Thu, Nov 05, 2015 at 02:16:15AM

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: > It's a perrenial request from our hardware PMU folks to be able to see the raw > values of the PMU registers. > > I think partly it's so that they can verify that perf is doing what they want, > and some of it is that they're

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 10:24 +0100, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: > > On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > > > > Second patch updates

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 13:17 +0530, Madhavan Srinivasan wrote: > On Friday 06 November 2015 08:28 AM, Sukadev Bhattiprolu wrote: > > Peter Zijlstra [pet...@infradead.org] wrote: > > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > > > > Second patch updates struct

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: > > > On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > >> Second patch updates struct arch_misc_reg for arch/powerpc with pmu > >>

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 03:34 PM, Michael Ellerman wrote: > On Fri, 2015-11-06 at 10:24 +0100, Peter Zijlstra wrote: >> On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: >>> On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: On Thu, Nov 05, 2015 at 02:16:15AM

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 03:55 PM, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: >> It's a perrenial request from our hardware PMU folks to be able to see the >> raw >> values of the PMU registers. >> >> I think partly it's so that they can verify

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 10:24 +0100, Peter Zijlstra wrote: > On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: > > On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > > > > Second patch updates

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, Nov 06, 2015 at 09:04:00PM +1100, Michael Ellerman wrote: > It's a perrenial request from our hardware PMU folks to be able to see the raw > values of the PMU registers. > > I think partly it's so that they can verify that perf is doing what they want, > and some of it is that they're

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, 2015-11-06 at 13:17 +0530, Madhavan Srinivasan wrote: > On Friday 06 November 2015 08:28 AM, Sukadev Bhattiprolu wrote: > > Peter Zijlstra [pet...@infradead.org] wrote: > > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > > > > Second patch updates struct

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Fri, Nov 06, 2015 at 12:57:17PM +0530, Madhavan Srinivasan wrote: > > > On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > >> Second patch updates struct arch_misc_reg for arch/powerpc with pmu > >>

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 08:28 AM, Sukadev Bhattiprolu wrote: > Peter Zijlstra [pet...@infradead.org] wrote: > | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu > registers > | > and adds offsetof

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thursday 05 November 2015 08:12 PM, Stephane Eranian wrote: > Hi, > > On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan > wrote: >> This patchset extend the perf sample regs infrastructure >> to include architecture specific regs. Current perf_sample_regs_intr >> exports only registers in

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: >> Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers >> and adds offsetof macro for the same. It extends perf_reg_value() >> to use reg

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

Peter Zijlstra [pet...@infradead.org] wrote: | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers | > and adds offsetof macro for the same. It extends perf_reg_value() | > to use reg idx to decide

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

Hi, On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan wrote: > > This patchset extend the perf sample regs infrastructure > to include architecture specific regs. Current perf_sample_regs_intr > exports only registers in the pt_regs to perf.data using > PERF_SAMPLE_REGS_INTR sample type. But

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers > and adds offsetof macro for the same. It extends perf_reg_value() > to use reg idx to decide on struct to return value from. Why; what's in those

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers > and adds offsetof macro for the same. It extends perf_reg_value() > to use reg idx to decide on struct to return value from. Why; what's in those

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

Peter Zijlstra [pet...@infradead.org] wrote: | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers | > and adds offsetof macro for the same. It extends perf_reg_value() | > to use reg idx to decide

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thursday 05 November 2015 08:12 PM, Stephane Eranian wrote: > Hi, > > On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan > wrote: >> This patchset extend the perf sample regs infrastructure >> to include architecture specific regs. Current perf_sample_regs_intr >>

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: >> Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers >> and adds offsetof macro for the same. It extends perf_reg_value() >> to use reg

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

On Friday 06 November 2015 08:28 AM, Sukadev Bhattiprolu wrote: > Peter Zijlstra [pet...@infradead.org] wrote: > | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu > registers > | > and adds offsetof

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

Hi, On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan wrote: > > This patchset extend the perf sample regs infrastructure > to include architecture specific regs. Current perf_sample_regs_intr > exports only registers in the pt_regs to perf.data using >