linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [RFC PATCH 00/15] ARM: perf: support multiple PMUs
       [not found] <4B9A4BAF850C914D8DED94776A2C477E0B853A2F@nasanexd01b.na.qualcomm.com>
@ 2011-08-22 19:04 ` Ashwin Chaugule
  2011-08-22 20:15   ` Will Deacon
  0 siblings, 1 reply; 5+ messages in thread
From: Ashwin Chaugule @ 2011-08-22 19:04 UTC (permalink / raw)
  To: linux-arm-kernel


>> -----Original Message-----
>> From: Mark Rutland
>> System (AKA nest or uncore) PMUs exist on devices which are not affine
>> to a single CPU. They usually cannot be directly associated with
>> individual tasks and are asynchronous with respect to the current
>> execution. Examples of devices which could have system PMUs include L2
>> cache controllers, GPUs and memory buses.
>>
>> The following patch series refactors the ARM PMU backend, enabling
>> new PMUs to reuse the existing code. This should allow for system PMUs
>> to be supported in future. Further work will be required to get perf to
>> fully understand system PMUs, but this provides something usable.
>>
>> The framework is intended to be used by system PMUs which hang off core
>> platform components (e.g. L2 cache, AXI bus). If a device is complex
>> enough or separate enough from core functionality to have its own
>> driver, it should implement its own PMU handling using the core perf
>> API directly.
>>
>> The first patch ("perf: provide PMU when initing events") is currently
>> sitting in the tip tree, but as it's required for event initialization
>> to function (and hence for the PMU to be usable), it's provided here
>> for convenience.
>>
>> The series is based on Will Deacon's perf-updates branch at:
>> 	git://linux-arm.org/linux-2.6-wd.git perf-updates
>>
>> An example driver using the framework (supporting the PMU present in
>> L220/PL310 level 2 cache controllers) can be found at:
>> 	git://linux-arm.org/linux-2.6-wd.git perf-l2x0-wip
>>
>> Any comments would be welcome.
>>
>> Thanks,
>> Mark.
>

Other than the comments on patch [05/15] and [08/15], this series looks
good to me !

Reviewed-by: Ashwin Chaugule <ashwinc@codeaurora.org>


-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [RFC PATCH 00/15] ARM: perf: support multiple PMUs
  2011-08-22 19:04 ` [RFC PATCH 00/15] ARM: perf: support multiple PMUs Ashwin Chaugule
@ 2011-08-22 20:15   ` Will Deacon
       [not found]     ` <CAMzctLkwJCrgKoeYET=OYcfOifKuTEWoWfcRgWts4Y6aspZEUA@mail.gmail.com>
  0 siblings, 1 reply; 5+ messages in thread
From: Will Deacon @ 2011-08-22 20:15 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Aug 22, 2011 at 08:04:50PM +0100, Ashwin Chaugule wrote:
> 
> Other than the comments on patch [05/15] and [08/15], this series looks
> good to me !
> 
> Reviewed-by: Ashwin Chaugule <ashwinc@codeaurora.org>

Cracking. Once -rc3 has landed I'll merge this into my perf-updates branch
(since some of my PMU refactoring will conflict with fixes going in for -rc3).

Thanks,

Will

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Fwd: [RFC PATCH 00/15] ARM: perf: support multiple PMUs
       [not found]     ` <CAMzctLkwJCrgKoeYET=OYcfOifKuTEWoWfcRgWts4Y6aspZEUA@mail.gmail.com>
@ 2012-06-13 11:43       ` Srinidhi Kasagar
  2012-06-13 17:02         ` Will Deacon
  0 siblings, 1 reply; 5+ messages in thread
From: Srinidhi Kasagar @ 2012-06-13 11:43 UTC (permalink / raw)
  To: linux-arm-kernel

Hello Marc/Will,

[...]

> Subject: [RFC PATCH 00/15] ARM: perf: support multiple PMUs
> ------------------------
> 
> From: Mark Rutland <mark.rutland at arm.com<mailto:mark.rutland@arm.com>>
> Date: Mon, Aug 15, 2011 at 7:12 PM
> To: linux-arm-kernel at lists.infradead.org

[...]

> System (AKA nest or uncore) PMUs exist on devices which are not affine
> to a single CPU. They usually cannot be directly associated with
> individual tasks and are asynchronous with respect to the current
> execution. Examples of devices which could have system PMUs include L2
> cache controllers, GPUs and memory buses.
> 
> The following patch series refactors the ARM PMU backend, enabling
> new PMUs to reuse the existing code. This should allow for system PMUs
> to be supported in future. Further work will be required to get perf to
> fully understand system PMUs, but this provides something usable.
> 
> The framework is intended to be used by system PMUs which hang off core
> platform components (e.g. L2 cache, AXI bus). If a device is complex
> enough or separate enough from core functionality to have its own
> driver, it should implement its own PMU handling using the core perf
> API directly.
> 
> The first patch ("perf: provide PMU when initing events") is currently
> sitting in the tip tree, but as it's required for event initialization
> to function (and hence for the PMU to be usable), it's provided here
> for convenience.
> 
> The series is based on Will Deacon's perf-updates branch at:
>        git://linux-arm.org/linux-2.6-wd.git perf-updates
> 
> An example driver using the framework (supporting the PMU present in
> L220/PL310 level 2 cache controllers) can be found at:
>        git://linux-arm.org/linux-2.6-wd.git perf-l2x0-wip
> 
Do you still maintain this tree? Looks it is empty. I'm wondering if
we can make use of this to instrument the PMU counters of L2/pl310.

/srinidhi 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Fwd: [RFC PATCH 00/15] ARM: perf: support multiple PMUs
  2012-06-13 11:43       ` Fwd: " Srinidhi Kasagar
@ 2012-06-13 17:02         ` Will Deacon
  2012-06-14  8:54           ` Srinidhi Kasagar
  0 siblings, 1 reply; 5+ messages in thread
From: Will Deacon @ 2012-06-13 17:02 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jun 13, 2012 at 12:43:27PM +0100, Srinidhi Kasagar wrote:
> > The series is based on Will Deacon's perf-updates branch at:
> >        git://linux-arm.org/linux-2.6-wd.git perf-updates
> > 
> > An example driver using the framework (supporting the PMU present in
> > L220/PL310 level 2 cache controllers) can be found at:
> >        git://linux-arm.org/linux-2.6-wd.git perf-l2x0-wip
> > 
> Do you still maintain this tree? Looks it is empty. I'm wondering if
> we can make use of this to instrument the PMU counters of L2/pl310.

There's an old branch here, but this stuff is pretty bit-rotted and needs
reworking as part of the big.LITTLE support:

http://git.kernel.org/?p=linux/kernel/git/will/linux.git;a=shortlog;h=refs/heads/perf/l2x0

Will

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Fwd: [RFC PATCH 00/15] ARM: perf: support multiple PMUs
  2012-06-13 17:02         ` Will Deacon
@ 2012-06-14  8:54           ` Srinidhi Kasagar
  0 siblings, 0 replies; 5+ messages in thread
From: Srinidhi Kasagar @ 2012-06-14  8:54 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jun 13, 2012 at 19:02:14 +0200, Will Deacon wrote:
> On Wed, Jun 13, 2012 at 12:43:27PM +0100, Srinidhi Kasagar wrote:
> > > The series is based on Will Deacon's perf-updates branch at:
> > >        git://linux-arm.org/linux-2.6-wd.git perf-updates
> > > 
> > > An example driver using the framework (supporting the PMU present in
> > > L220/PL310 level 2 cache controllers) can be found at:
> > >        git://linux-arm.org/linux-2.6-wd.git perf-l2x0-wip
> > > 
> > Do you still maintain this tree? Looks it is empty. I'm wondering if
> > we can make use of this to instrument the PMU counters of L2/pl310.
> 
> There's an old branch here, but this stuff is pretty bit-rotted and needs
> reworking as part of the big.LITTLE support:
> 
> http://git.kernel.org/?p=linux/kernel/git/will/linux.git;a=shortlog;h=refs/heads/perf/l2x0

Thanks, will look at it.

/srinidhi

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-06-14  8:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <4B9A4BAF850C914D8DED94776A2C477E0B853A2F@nasanexd01b.na.qualcomm.com>
2011-08-22 19:04 ` [RFC PATCH 00/15] ARM: perf: support multiple PMUs Ashwin Chaugule
2011-08-22 20:15   ` Will Deacon
     [not found]     ` <CAMzctLkwJCrgKoeYET=OYcfOifKuTEWoWfcRgWts4Y6aspZEUA@mail.gmail.com>
2012-06-13 11:43       ` Fwd: " Srinidhi Kasagar
2012-06-13 17:02         ` Will Deacon
2012-06-14  8:54           ` Srinidhi Kasagar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).