Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-12 Thread Thomas Gleixner
On Tue, 12 Nov 2013, Vinayak Kale wrote: > On Mon, Nov 11, 2013 at 4:14 PM, Will Deacon wrote: > > cleaner from a user perspective, I always get scared when I see > > 'desc->status_use_accessors' since it tends to incur the wrath of tglx :) Nah, if it uses accessors, then it's fine. > > That

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-12 Thread Thomas Gleixner
On Tue, 12 Nov 2013, Vinayak Kale wrote: On Mon, Nov 11, 2013 at 4:14 PM, Will Deacon will.dea...@arm.com wrote: cleaner from a user perspective, I always get scared when I see 'desc-status_use_accessors' since it tends to incur the wrath of tglx :) Nah, if it uses accessors, then it's fine.

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Vinayak Kale
On Mon, Nov 11, 2013 at 4:14 PM, Will Deacon wrote: > On Sat, Nov 09, 2013 at 01:04:23AM +, Stephen Boyd wrote: >> On 11/06/13 04:07, Vinayak Kale wrote: >> > This patch series adds support to handle interrupt >> > registration/deregistration >> > in arm64 pmu driver when pmu interrupt type

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Ganapat Rao P Kulkarni
Will, armv8pmu_read_counter() and armv8pmu_write_counter() treats cycle counter as 32 bit counter. AFAIK, the cycle counter on ARM64 is 64 bit and overflow interrupt comes either 32 bit changes or when 64 bit flips. and HRM says, ARM deprecates use of PMCR_EL0.LC = 0 => cycle counter of 64 bit

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Will Deacon
On Sat, Nov 09, 2013 at 01:04:23AM +, Stephen Boyd wrote: > On 11/06/13 04:07, Vinayak Kale wrote: > > This patch series adds support to handle interrupt > > registration/deregistration > > in arm64 pmu driver when pmu interrupt type is percpu. > > > > Patches in this patch series were

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Vinayak Kale
On Sat, Nov 9, 2013 at 6:34 AM, Stephen Boyd wrote: > On 11/06/13 04:07, Vinayak Kale wrote: >> This patch series adds support to handle interrupt >> registration/deregistration >> in arm64 pmu driver when pmu interrupt type is percpu. >> >> Patches in this patch series were previously sent out

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Vinayak Kale
On Sat, Nov 9, 2013 at 6:34 AM, Stephen Boyd sb...@codeaurora.org wrote: On 11/06/13 04:07, Vinayak Kale wrote: This patch series adds support to handle interrupt registration/deregistration in arm64 pmu driver when pmu interrupt type is percpu. Patches in this patch series were previously

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Will Deacon
On Sat, Nov 09, 2013 at 01:04:23AM +, Stephen Boyd wrote: On 11/06/13 04:07, Vinayak Kale wrote: This patch series adds support to handle interrupt registration/deregistration in arm64 pmu driver when pmu interrupt type is percpu. Patches in this patch series were previously sent

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Ganapat Rao P Kulkarni
Will, armv8pmu_read_counter() and armv8pmu_write_counter() treats cycle counter as 32 bit counter. AFAIK, the cycle counter on ARM64 is 64 bit and overflow interrupt comes either 32 bit changes or when 64 bit flips. and HRM says, ARM deprecates use of PMCR_EL0.LC = 0 = cycle counter of 64 bit

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-11 Thread Vinayak Kale
On Mon, Nov 11, 2013 at 4:14 PM, Will Deacon will.dea...@arm.com wrote: On Sat, Nov 09, 2013 at 01:04:23AM +, Stephen Boyd wrote: On 11/06/13 04:07, Vinayak Kale wrote: This patch series adds support to handle interrupt registration/deregistration in arm64 pmu driver when pmu

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-08 Thread Stephen Boyd
On 11/06/13 04:07, Vinayak Kale wrote: > This patch series adds support to handle interrupt registration/deregistration > in arm64 pmu driver when pmu interrupt type is percpu. > > Patches in this patch series were previously sent out as separate patches [1]. > This patch series incorporates

Re: [PATCH 0/2] genirq: arm64: perf: support for percpu pmu interrupt

2013-11-08 Thread Stephen Boyd
On 11/06/13 04:07, Vinayak Kale wrote: This patch series adds support to handle interrupt registration/deregistration in arm64 pmu driver when pmu interrupt type is percpu. Patches in this patch series were previously sent out as separate patches [1]. This patch series incorporates