* [RESEND PATCH v3] ARM: tegra124: pmu support @ 2015-07-13 17:35 Kyle Huey 2015-07-15 9:35 ` Mark Rutland 2015-07-17 8:59 ` Thierry Reding 0 siblings, 2 replies; 6+ messages in thread From: Kyle Huey @ 2015-07-13 17:35 UTC (permalink / raw) To: linux-arm-kernel This patch modifies the device tree for tegra124 based devices to enable the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM DP-06905-001_v03p. This patch was tested on a Jetson TK1. Updated for proper ordering and to add interrupt-affinity values. Signed-off-by: Kyle Huey <khuey@kylehuey.com> --- arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi index 13cc7ca..de07d7e 100644 --- a/arch/arm/boot/dts/tegra124.dtsi +++ b/arch/arm/boot/dts/tegra124.dtsi @@ -918,31 +918,40 @@ #address-cells = <1>; #size-cells = <0>; - cpu at 0 { + A15_0: cpu at 0 { device_type = "cpu"; compatible = "arm,cortex-a15"; reg = <0>; }; - cpu at 1 { + A15_1: cpu at 1 { device_type = "cpu"; compatible = "arm,cortex-a15"; reg = <1>; }; - cpu at 2 { + A15_2: cpu at 2 { device_type = "cpu"; compatible = "arm,cortex-a15"; reg = <2>; }; - cpu at 3 { + A15_3: cpu at 3 { device_type = "cpu"; compatible = "arm,cortex-a15"; reg = <3>; }; }; + pmu { + compatible = "arm,cortex-a15-pmu"; + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; + }; + thermal-zones { cpu { polling-delay-passive = <1000>; -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [RESEND PATCH v3] ARM: tegra124: pmu support 2015-07-13 17:35 [RESEND PATCH v3] ARM: tegra124: pmu support Kyle Huey @ 2015-07-15 9:35 ` Mark Rutland 2015-07-17 7:58 ` Jon Hunter 2015-07-17 8:59 ` Thierry Reding 1 sibling, 1 reply; 6+ messages in thread From: Mark Rutland @ 2015-07-15 9:35 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jul 13, 2015 at 06:35:45PM +0100, Kyle Huey wrote: > This patch modifies the device tree for tegra124 based devices to enable > the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM > DP-06905-001_v03p. This patch was tested on a Jetson TK1. > > Updated for proper ordering and to add interrupt-affinity values. > > Signed-off-by: Kyle Huey <khuey@kylehuey.com> This looks sane to me, and as you've tested it the values seem to be valid: Acked-by: Mark Rutland <mark.rutland@arm.com> Mark. > --- > arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi > index 13cc7ca..de07d7e 100644 > --- a/arch/arm/boot/dts/tegra124.dtsi > +++ b/arch/arm/boot/dts/tegra124.dtsi > @@ -918,31 +918,40 @@ > #address-cells = <1>; > #size-cells = <0>; > > - cpu at 0 { > + A15_0: cpu at 0 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <0>; > }; > > - cpu at 1 { > + A15_1: cpu at 1 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <1>; > }; > > - cpu at 2 { > + A15_2: cpu at 2 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <2>; > }; > > - cpu at 3 { > + A15_3: cpu at 3 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <3>; > }; > }; > > + pmu { > + compatible = "arm,cortex-a15-pmu"; > + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; > + }; > + > thermal-zones { > cpu { > polling-delay-passive = <1000>; > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [RESEND PATCH v3] ARM: tegra124: pmu support 2015-07-15 9:35 ` Mark Rutland @ 2015-07-17 7:58 ` Jon Hunter 0 siblings, 0 replies; 6+ messages in thread From: Jon Hunter @ 2015-07-17 7:58 UTC (permalink / raw) To: linux-arm-kernel On 15/07/15 10:35, Mark Rutland wrote: > On Mon, Jul 13, 2015 at 06:35:45PM +0100, Kyle Huey wrote: >> This patch modifies the device tree for tegra124 based devices to enable >> the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM >> DP-06905-001_v03p. This patch was tested on a Jetson TK1. >> >> Updated for proper ordering and to add interrupt-affinity values. >> >> Signed-off-by: Kyle Huey <khuey@kylehuey.com> > > This looks sane to me, and as you've tested it the values seem to be > valid: > > Acked-by: Mark Rutland <mark.rutland@arm.com> FWIW ... Acked-by: Jon Hunter <jonathanh@nvidia.com> Stephen, Thierry, Alex, Can we pick this up now? Jon >> --- >> arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- >> 1 file changed, 13 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi >> index 13cc7ca..de07d7e 100644 >> --- a/arch/arm/boot/dts/tegra124.dtsi >> +++ b/arch/arm/boot/dts/tegra124.dtsi >> @@ -918,31 +918,40 @@ >> #address-cells = <1>; >> #size-cells = <0>; >> >> - cpu at 0 { >> + A15_0: cpu at 0 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <0>; >> }; >> >> - cpu at 1 { >> + A15_1: cpu at 1 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <1>; >> }; >> >> - cpu at 2 { >> + A15_2: cpu at 2 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <2>; >> }; >> >> - cpu at 3 { >> + A15_3: cpu at 3 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <3>; >> }; >> }; >> >> + pmu { >> + compatible = "arm,cortex-a15-pmu"; >> + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; >> + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; >> + }; >> + >> thermal-zones { >> cpu { >> polling-delay-passive = <1000>; >> -- >> 1.9.1 >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [RESEND PATCH v3] ARM: tegra124: pmu support 2015-07-13 17:35 [RESEND PATCH v3] ARM: tegra124: pmu support Kyle Huey 2015-07-15 9:35 ` Mark Rutland @ 2015-07-17 8:59 ` Thierry Reding 2015-07-18 13:54 ` Kyle Huey 1 sibling, 1 reply; 6+ messages in thread From: Thierry Reding @ 2015-07-17 8:59 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jul 13, 2015 at 10:35:45AM -0700, Kyle Huey wrote: > This patch modifies the device tree for tegra124 based devices to enable > the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM > DP-06905-001_v03p. This patch was tested on a Jetson TK1. > > Updated for proper ordering and to add interrupt-affinity values. > > Signed-off-by: Kyle Huey <khuey@kylehuey.com> > --- > arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) Is there any way to test this? What are the effects of adding this? Does it enable using perf for profiling? > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi > index 13cc7ca..de07d7e 100644 > --- a/arch/arm/boot/dts/tegra124.dtsi > +++ b/arch/arm/boot/dts/tegra124.dtsi > @@ -918,31 +918,40 @@ > #address-cells = <1>; > #size-cells = <0>; > > - cpu at 0 { > + A15_0: cpu at 0 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <0>; > }; > > - cpu at 1 { > + A15_1: cpu at 1 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <1>; > }; > > - cpu at 2 { > + A15_2: cpu at 2 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <2>; > }; > > - cpu at 3 { > + A15_3: cpu at 3 { > device_type = "cpu"; > compatible = "arm,cortex-a15"; > reg = <3>; > }; > }; > > + pmu { > + compatible = "arm,cortex-a15-pmu"; > + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; These labels look somewhat artificial to me, perhaps we could do something like the following instead? interrupt-affinity = <&{/cpus/cpu@0}>, ...; That's slightly more obvious and avoids the need to "invent" labels for the CPUs. No need to respin, I can fix that up when applying if nobody objects to using the alternative notation. Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150717/ac176d0d/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [RESEND PATCH v3] ARM: tegra124: pmu support 2015-07-17 8:59 ` Thierry Reding @ 2015-07-18 13:54 ` Kyle Huey 2015-07-27 16:46 ` Kyle Huey 0 siblings, 1 reply; 6+ messages in thread From: Kyle Huey @ 2015-07-18 13:54 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 17, 2015 at 4:59 PM, Thierry Reding <thierry.reding@gmail.com> wrote: > On Mon, Jul 13, 2015 at 10:35:45AM -0700, Kyle Huey wrote: >> This patch modifies the device tree for tegra124 based devices to enable >> the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM >> DP-06905-001_v03p. This patch was tested on a Jetson TK1. >> >> Updated for proper ordering and to add interrupt-affinity values. >> >> Signed-off-by: Kyle Huey <khuey@kylehuey.com> >> --- >> arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- >> 1 file changed, 13 insertions(+), 4 deletions(-) > > Is there any way to test this? What are the effects of adding this? Yes. This enables the ARM PMU driver for the Cortex A15, which allows one to use hardware performance counters via the perf_event_open API. For a simple test program, see https://github.com/khuey/perf-counter-test/. Without this patch, the perf_event_open syscall will fail. With this patch, the program will print out the performance counter value for each iteration of the loop. (IIRC on the A15 the branch counter was removed, so you may want to replace 0xD with 0x8 which counts instructions executed if you want to see a non-zero number there). You also will see a message about the PMU in the kernel log at startup after applying this patch. I have also tested this extensively (including the interrupt features of the PMU) on a more complex program. > Does it enable using perf for profiling? I have not tested it, but I believe you can use perf without this patch if you do not use features that require hardware performance counter support. This patch would enable those features. >> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi >> index 13cc7ca..de07d7e 100644 >> --- a/arch/arm/boot/dts/tegra124.dtsi >> +++ b/arch/arm/boot/dts/tegra124.dtsi >> @@ -918,31 +918,40 @@ >> #address-cells = <1>; >> #size-cells = <0>; >> >> - cpu at 0 { >> + A15_0: cpu at 0 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <0>; >> }; >> >> - cpu at 1 { >> + A15_1: cpu at 1 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <1>; >> }; >> >> - cpu at 2 { >> + A15_2: cpu at 2 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <2>; >> }; >> >> - cpu at 3 { >> + A15_3: cpu at 3 { >> device_type = "cpu"; >> compatible = "arm,cortex-a15"; >> reg = <3>; >> }; >> }; >> >> + pmu { >> + compatible = "arm,cortex-a15-pmu"; >> + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; >> + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; > > These labels look somewhat artificial to me, perhaps we could do > something like the following instead? > > interrupt-affinity = <&{/cpus/cpu@0}>, ...; > > That's slightly more obvious and avoids the need to "invent" labels for > the CPUs. > > No need to respin, I can fix that up when applying if nobody objects to > using the alternative notation. > > Thierry I have no objections. I was not aware that the device tree syntax supported that. FWIW I cargo-culted my way to victory from vexpress-v2p-ca9.dts here. - Kyle ^ permalink raw reply [flat|nested] 6+ messages in thread
* [RESEND PATCH v3] ARM: tegra124: pmu support 2015-07-18 13:54 ` Kyle Huey @ 2015-07-27 16:46 ` Kyle Huey 0 siblings, 0 replies; 6+ messages in thread From: Kyle Huey @ 2015-07-27 16:46 UTC (permalink / raw) To: linux-arm-kernel On Sat, Jul 18, 2015 at 6:54 AM, Kyle Huey <me@kylehuey.com> wrote: > On Fri, Jul 17, 2015 at 4:59 PM, Thierry Reding > <thierry.reding@gmail.com> wrote: >> On Mon, Jul 13, 2015 at 10:35:45AM -0700, Kyle Huey wrote: >>> This patch modifies the device tree for tegra124 based devices to enable >>> the Cortex A15 PMU. The interrupt numbers are taken from NVIDIA TRM >>> DP-06905-001_v03p. This patch was tested on a Jetson TK1. >>> >>> Updated for proper ordering and to add interrupt-affinity values. >>> >>> Signed-off-by: Kyle Huey <khuey@kylehuey.com> >>> --- >>> arch/arm/boot/dts/tegra124.dtsi | 17 +++++++++++++---- >>> 1 file changed, 13 insertions(+), 4 deletions(-) >> >> Is there any way to test this? What are the effects of adding this? > > Yes. This enables the ARM PMU driver for the Cortex A15, which allows > one to use hardware performance counters via the perf_event_open API. > For a simple test program, see > https://github.com/khuey/perf-counter-test/. Without this patch, the > perf_event_open syscall will fail. With this patch, the program will > print out the performance counter value for each iteration of the > loop. (IIRC on the A15 the branch counter was removed, so you may want > to replace 0xD with 0x8 which counts instructions executed if you want > to see a non-zero number there). You also will see a message about > the PMU in the kernel log at startup after applying this patch. > > I have also tested this extensively (including the interrupt features > of the PMU) on a more complex program. > >> Does it enable using perf for profiling? > > I have not tested it, but I believe you can use perf without this > patch if you do not use features that require hardware performance > counter support. This patch would enable those features. > >>> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi >>> index 13cc7ca..de07d7e 100644 >>> --- a/arch/arm/boot/dts/tegra124.dtsi >>> +++ b/arch/arm/boot/dts/tegra124.dtsi >>> @@ -918,31 +918,40 @@ >>> #address-cells = <1>; >>> #size-cells = <0>; >>> >>> - cpu at 0 { >>> + A15_0: cpu at 0 { >>> device_type = "cpu"; >>> compatible = "arm,cortex-a15"; >>> reg = <0>; >>> }; >>> >>> - cpu at 1 { >>> + A15_1: cpu at 1 { >>> device_type = "cpu"; >>> compatible = "arm,cortex-a15"; >>> reg = <1>; >>> }; >>> >>> - cpu at 2 { >>> + A15_2: cpu at 2 { >>> device_type = "cpu"; >>> compatible = "arm,cortex-a15"; >>> reg = <2>; >>> }; >>> >>> - cpu at 3 { >>> + A15_3: cpu at 3 { >>> device_type = "cpu"; >>> compatible = "arm,cortex-a15"; >>> reg = <3>; >>> }; >>> }; >>> >>> + pmu { >>> + compatible = "arm,cortex-a15-pmu"; >>> + interrupts = <GIC_SPI 144 IRQ_TYPE_LEVEL_HIGH>, >>> + <GIC_SPI 145 IRQ_TYPE_LEVEL_HIGH>, >>> + <GIC_SPI 146 IRQ_TYPE_LEVEL_HIGH>, >>> + <GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>; >>> + interrupt-affinity = <&A15_0>, <&A15_1>, <&A15_2>, <&A15_3>; >> >> These labels look somewhat artificial to me, perhaps we could do >> something like the following instead? >> >> interrupt-affinity = <&{/cpus/cpu@0}>, ...; >> >> That's slightly more obvious and avoids the need to "invent" labels for >> the CPUs. >> >> No need to respin, I can fix that up when applying if nobody objects to >> using the alternative notation. >> >> Thierry > > I have no objections. I was not aware that the device tree syntax > supported that. FWIW I cargo-culted my way to victory from > vexpress-v2p-ca9.dts here. > > - Kyle Anything else I can do to help move this along? - Kyle ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-07-27 16:46 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-07-13 17:35 [RESEND PATCH v3] ARM: tegra124: pmu support Kyle Huey 2015-07-15 9:35 ` Mark Rutland 2015-07-17 7:58 ` Jon Hunter 2015-07-17 8:59 ` Thierry Reding 2015-07-18 13:54 ` Kyle Huey 2015-07-27 16:46 ` Kyle Huey
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).