* [GIT PULL] ARM perf updates for 4.1 @ 2015-03-27 17:22 Will Deacon 2015-04-07 15:05 ` Geert Uytterhoeven 0 siblings, 1 reply; 7+ messages in thread From: Will Deacon @ 2015-03-27 17:22 UTC (permalink / raw) To: linux-arm-kernel Hi Russell, Please can you pull the following ARM perf updates for 4.1? The highlight is the addition of Qualcomm Scorpion support from Stephen, but there's also a small fix and a device-tree parsing update to allow for explicit specification of PMU interrupt affinity. Thanks, Will --->8 The following changes since commit 06e5801b8cb3fc057d88cb4dc03c0b64b2744cda: Linux 4.0-rc4 (2015-03-15 17:38:20 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git for-rmk/perf for you to fetch changes up to 9fd85eb502a78bd812db58bd1f668b2a06ee30a5: ARM: pmu: add support for interrupt-affinity property (2015-03-24 15:07:57 +0000) ---------------------------------------------------------------- Stephen Boyd (3): ARM: perf: Preparatory work for Scorpion PMU support ARM: perf: Only reset PMxEVCNTCR registers on reset ARM: perf: Add support for Scorpion PMUs Suzuki K. Poulose (1): ARM: perf: reject groups spanning multiple hardware PMUs Will Deacon (1): ARM: pmu: add support for interrupt-affinity property Documentation/devicetree/bindings/arm/pmu.txt | 2 + arch/arm/include/asm/pmu.h | 1 + arch/arm/kernel/perf_event.c | 21 +- arch/arm/kernel/perf_event_cpu.c | 71 +++- arch/arm/kernel/perf_event_v7.c | 525 +++++++++++++++++++++++--- 5 files changed, 548 insertions(+), 72 deletions(-) ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-03-27 17:22 [GIT PULL] ARM perf updates for 4.1 Will Deacon @ 2015-04-07 15:05 ` Geert Uytterhoeven 2015-04-13 9:10 ` Will Deacon 0 siblings, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2015-04-07 15:05 UTC (permalink / raw) To: linux-arm-kernel Hi Will, On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: > Will Deacon (1): > ARM: pmu: add support for interrupt-affinity property As most DTSes lack this property, we now get scary warnings: CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] BTW, shouldn't the DT update go in first, or together with the code update? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-04-07 15:05 ` Geert Uytterhoeven @ 2015-04-13 9:10 ` Will Deacon 2015-04-13 9:28 ` Maxime Ripard 2015-04-13 9:30 ` Geert Uytterhoeven 0 siblings, 2 replies; 7+ messages in thread From: Will Deacon @ 2015-04-13 9:10 UTC (permalink / raw) To: linux-arm-kernel On Tue, Apr 07, 2015 at 04:05:40PM +0100, Geert Uytterhoeven wrote: > On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: > > Will Deacon (1): > > ARM: pmu: add support for interrupt-affinity property > > As most DTSes lack this property, we now get scary warnings: > > CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] That's a harmless warning (i.e. perf will `work' as before), but I'd like to print something to say that we didn't find the property. > BTW, shouldn't the DT update go in first, or together with the code update? It's going via the arm64 tree (I had to choose one of the trees for the binding, since they both arm and arm64 use it). Will ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-04-13 9:10 ` Will Deacon @ 2015-04-13 9:28 ` Maxime Ripard 2015-04-13 10:21 ` Will Deacon 2015-04-13 9:30 ` Geert Uytterhoeven 1 sibling, 1 reply; 7+ messages in thread From: Maxime Ripard @ 2015-04-13 9:28 UTC (permalink / raw) To: linux-arm-kernel Hi Will, On Mon, Apr 13, 2015 at 10:10:12AM +0100, Will Deacon wrote: > On Tue, Apr 07, 2015 at 04:05:40PM +0100, Geert Uytterhoeven wrote: > > On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: > > > Will Deacon (1): > > > ARM: pmu: add support for interrupt-affinity property > > > > As most DTSes lack this property, we now get scary warnings: > > > > CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] > > That's a harmless warning (i.e. perf will `work' as before), but I'd like to > print something to say that we didn't find the property. Shouldn't we have a warning only if that property makes some kind of sense? I mean, I agree on the fact that we want this property if this is an SPI, but if it is a PPI, it doesn't make any sense to have this property, and this is very well documented in the binding documentation. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150413/76f8ce8b/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-04-13 9:28 ` Maxime Ripard @ 2015-04-13 10:21 ` Will Deacon 0 siblings, 0 replies; 7+ messages in thread From: Will Deacon @ 2015-04-13 10:21 UTC (permalink / raw) To: linux-arm-kernel On Mon, Apr 13, 2015 at 10:28:12AM +0100, Maxime Ripard wrote: > On Mon, Apr 13, 2015 at 10:10:12AM +0100, Will Deacon wrote: > > On Tue, Apr 07, 2015 at 04:05:40PM +0100, Geert Uytterhoeven wrote: > > > On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: > > > > Will Deacon (1): > > > > ARM: pmu: add support for interrupt-affinity property > > > > > > As most DTSes lack this property, we now get scary warnings: > > > > > > CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] > > > > That's a harmless warning (i.e. perf will `work' as before), but I'd like to > > print something to say that we didn't find the property. > > Shouldn't we have a warning only if that property makes some kind of > sense? > > I mean, I agree on the fact that we want this property if this is an > SPI, but if it is a PPI, it doesn't make any sense to have this > property, and this is very well documented in the binding > documentation. Yes, that's a good point; for PPIs, the warning is nonsensical. I'll cook a fix. Cheers, Will ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-04-13 9:10 ` Will Deacon 2015-04-13 9:28 ` Maxime Ripard @ 2015-04-13 9:30 ` Geert Uytterhoeven 2015-04-13 9:33 ` Will Deacon 1 sibling, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2015-04-13 9:30 UTC (permalink / raw) To: linux-arm-kernel Hi Will, On Mon, Apr 13, 2015 at 11:10 AM, Will Deacon <will.deacon@arm.com> wrote: > On Tue, Apr 07, 2015 at 04:05:40PM +0100, Geert Uytterhoeven wrote: >> On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: >> > Will Deacon (1): >> > ARM: pmu: add support for interrupt-affinity property >> >> As most DTSes lack this property, we now get scary warnings: >> >> CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] > > That's a harmless warning (i.e. perf will `work' as before), but I'd like to > print something to say that we didn't find the property. > >> BTW, shouldn't the DT update go in first, or together with the code update? > > It's going via the arm64 tree (I had to choose one of the trees for the > binding, since they both arm and arm64 use it). OK. Will start to pull arm64 too into renesas-drivers... BTW, time to start unifying arm32 and arm64? ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM perf updates for 4.1 2015-04-13 9:30 ` Geert Uytterhoeven @ 2015-04-13 9:33 ` Will Deacon 0 siblings, 0 replies; 7+ messages in thread From: Will Deacon @ 2015-04-13 9:33 UTC (permalink / raw) To: linux-arm-kernel On Mon, Apr 13, 2015 at 10:30:22AM +0100, Geert Uytterhoeven wrote: > On Mon, Apr 13, 2015 at 11:10 AM, Will Deacon <will.deacon@arm.com> wrote: > > On Tue, Apr 07, 2015 at 04:05:40PM +0100, Geert Uytterhoeven wrote: > >> On Fri, Mar 27, 2015 at 6:22 PM, Will Deacon <will.deacon@arm.com> wrote: > >> > Will Deacon (1): > >> > ARM: pmu: add support for interrupt-affinity property > >> > >> As most DTSes lack this property, we now get scary warnings: > >> > >> CPU PMU: Failed to parse <no-node>/interrupt-affinity[0] > > > > That's a harmless warning (i.e. perf will `work' as before), but I'd like to > > print something to say that we didn't find the property. > > > >> BTW, shouldn't the DT update go in first, or together with the code update? > > > > It's going via the arm64 tree (I had to choose one of the trees for the > > binding, since they both arm and arm64 use it). > > OK. Will start to pull arm64 too into renesas-drivers... > > BTW, time to start unifying arm32 and arm64? ;-) We're already working on unifying the perf bits (the major duplication imo) by moving the bulk of it into drivers. Stay tuned! Will ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-04-13 10:21 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-03-27 17:22 [GIT PULL] ARM perf updates for 4.1 Will Deacon 2015-04-07 15:05 ` Geert Uytterhoeven 2015-04-13 9:10 ` Will Deacon 2015-04-13 9:28 ` Maxime Ripard 2015-04-13 10:21 ` Will Deacon 2015-04-13 9:30 ` Geert Uytterhoeven 2015-04-13 9:33 ` Will Deacon
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.