* [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE [not found] <20231016084015.400031271@linuxfoundation.org> @ 2023-10-16 8:39 ` Greg Kroah-Hartman 2023-10-16 16:05 ` Thorsten Leemhuis 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2023-10-16 8:39 UTC (permalink / raw) To: stable Cc: Greg Kroah-Hartman, patches, Catalin Marinas, Will Deacon, Mark Rutland, linux-arm-kernel, linux-kernel, Anshuman Khandual, Sasha Levin 6.5-stable review patch. If anyone has any objections, please let me know. ------------------ From: Anshuman Khandual <anshuman.khandual@arm.com> [ Upstream commit 1aa3d0274a4aac338ee45a3dfc3b17c944bcc2bc ] ACPI TRBE does not have a HID for identification which could create and add a platform device into the platform bus. Also without a platform device, it cannot be probed and bound to a platform driver. This creates a dummy platform device for TRBE after ascertaining that ACPI provides required interrupts uniformly across all cpus on the system. This device gets created inside drivers/perf/arm_pmu_acpi.c to accommodate TRBE being built as a module. Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com> Link: https://lore.kernel.org/r/20230817055405.249630-3-anshuman.khandual@arm.com Signed-off-by: Will Deacon <will@kernel.org> Stable-dep-of: 4785aa802853 ("cpuidle, ACPI: Evaluate LPI arch_flags for broadcast timer") Signed-off-by: Sasha Levin <sashal@kernel.org> --- arch/arm64/include/asm/acpi.h | 3 +++ drivers/perf/arm_pmu_acpi.c | 35 +++++++++++++++++++++++++++++++++++ include/linux/perf/arm_pmu.h | 1 + 3 files changed, 39 insertions(+) diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h index bd68e1b7f29f3..4d537d56eb847 100644 --- a/arch/arm64/include/asm/acpi.h +++ b/arch/arm64/include/asm/acpi.h @@ -42,6 +42,9 @@ #define ACPI_MADT_GICC_SPE (offsetof(struct acpi_madt_generic_interrupt, \ spe_interrupt) + sizeof(u16)) +#define ACPI_MADT_GICC_TRBE (offsetof(struct acpi_madt_generic_interrupt, \ + trbe_interrupt) + sizeof(u16)) + /* Basic configuration for ACPI */ #ifdef CONFIG_ACPI pgprot_t __acpi_get_mem_attribute(phys_addr_t addr); diff --git a/drivers/perf/arm_pmu_acpi.c b/drivers/perf/arm_pmu_acpi.c index 90815ad762ebc..8baeeb369118e 100644 --- a/drivers/perf/arm_pmu_acpi.c +++ b/drivers/perf/arm_pmu_acpi.c @@ -139,6 +139,40 @@ static inline void arm_spe_acpi_register_device(void) } #endif /* CONFIG_ARM_SPE_PMU */ +#if IS_ENABLED(CONFIG_CORESIGHT_TRBE) +static struct resource trbe_resources[] = { + { + /* irq */ + .flags = IORESOURCE_IRQ, + } +}; + +static struct platform_device trbe_dev = { + .name = ARMV8_TRBE_PDEV_NAME, + .id = -1, + .resource = trbe_resources, + .num_resources = ARRAY_SIZE(trbe_resources) +}; + +static u16 arm_trbe_parse_gsi(struct acpi_madt_generic_interrupt *gicc) +{ + return gicc->trbe_interrupt; +} + +static void arm_trbe_acpi_register_device(void) +{ + int ret = arm_acpi_register_pmu_device(&trbe_dev, ACPI_MADT_GICC_TRBE, + arm_trbe_parse_gsi); + if (ret) + pr_warn("ACPI: TRBE: Unable to register device\n"); +} +#else +static inline void arm_trbe_acpi_register_device(void) +{ + +} +#endif /* CONFIG_CORESIGHT_TRBE */ + static int arm_pmu_acpi_parse_irqs(void) { int irq, cpu, irq_cpu, err; @@ -374,6 +408,7 @@ static int arm_pmu_acpi_init(void) return 0; arm_spe_acpi_register_device(); + arm_trbe_acpi_register_device(); return 0; } diff --git a/include/linux/perf/arm_pmu.h b/include/linux/perf/arm_pmu.h index a0801f68762bf..143fbc10ecfe0 100644 --- a/include/linux/perf/arm_pmu.h +++ b/include/linux/perf/arm_pmu.h @@ -187,5 +187,6 @@ void armpmu_free_irq(int irq, int cpu); #endif /* CONFIG_ARM_PMU */ #define ARMV8_SPE_PDEV_NAME "arm,spe-v1" +#define ARMV8_TRBE_PDEV_NAME "arm,trbe" #endif /* __ARM_PMU_H__ */ -- 2.40.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE 2023-10-16 8:39 ` [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE Greg Kroah-Hartman @ 2023-10-16 16:05 ` Thorsten Leemhuis 2023-10-16 16:52 ` Greg Kroah-Hartman 0 siblings, 1 reply; 4+ messages in thread From: Thorsten Leemhuis @ 2023-10-16 16:05 UTC (permalink / raw) To: Greg Kroah-Hartman, stable Cc: patches, Catalin Marinas, Will Deacon, Mark Rutland, linux-arm-kernel, linux-kernel, Anshuman Khandual, Sasha Levin On 16.10.23 10:39, Greg Kroah-Hartman wrote: > 6.5-stable review patch. If anyone has any objections, please let me know. > > ------------------ > > From: Anshuman Khandual <anshuman.khandual@arm.com> > > [ Upstream commit 1aa3d0274a4aac338ee45a3dfc3b17c944bcc2bc ] > > ACPI TRBE does not have a HID for identification which could create and add > a platform device into the platform bus. Also without a platform device, it > cannot be probed and bound to a platform driver. > > This creates a dummy platform device for TRBE after ascertaining that ACPI > provides required interrupts uniformly across all cpus on the system. This > device gets created inside drivers/perf/arm_pmu_acpi.c to accommodate TRBE > being built as a module. > > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Will Deacon <will@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com> > Link: https://lore.kernel.org/r/20230817055405.249630-3-anshuman.khandual@arm.com > Signed-off-by: Will Deacon <will@kernel.org> > Stable-dep-of: 4785aa802853 ("cpuidle, ACPI: Evaluate LPI arch_flags for broadcast timer") > Signed-off-by: Sasha Levin <sashal@kernel.org> Got a compiler error when using Fedora's stable kernel config and compiling 6.5.8-rc1 for ARM64; not totally sure, but from the error msg I suspect it's caused by the change quoted above. """ + /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -mbranch-protection=standard -fasynchronous-unwind-tables -fstack-clash-protection' 'HOSTLDFLAGS=-Wl,-z,relro -Wl,--as-needed -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--build-id=sha1 -specs=/usr/lib/rpm/redhat/redhat-package-notes' ARCH=arm64 'KCFLAGS= ' WITH_GCOV=0 -j4 vmlinuz.efi drivers/perf/arm_pmu_acpi.c: In function 'arm_trbe_acpi_register_device': drivers/perf/arm_pmu_acpi.c:164:19: error: implicit declaration of function 'arm_acpi_register_pmu_device'; did you mean 'arm_spe_acpi_register_device'? [-Werror=implicit-function-declaration] 164 | int ret = arm_acpi_register_pmu_device(&trbe_dev, ACPI_MADT_GICC_TRBE, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ | arm_spe_acpi_register_device cc1: some warnings being treated as errors make[4]: *** [scripts/Makefile.build:243: drivers/perf/arm_pmu_acpi.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:480: drivers/perf] Error 2 make[3]: *** Waiting for unfinished jobs.... make[2]: *** [scripts/Makefile.build:480: drivers] Error 2 make[1]: *** [/builddir/build/BUILD/kernel-6.5.8-rc1/linux-6.5.8-0.rc1.150.vanilla.fc37.aarch64/Makefile:2036: .] Error 2 make: *** [Makefile:236: __sub-make] Error 2 """ Full build log: https://copr-be.cloud.fedoraproject.org/results/@kernel-vanilla/stable-rc/fedora-37-aarch64/06529454-stablerc-stablerc-releases/builder-live.log.gz Ciao, Thorsten > --- > arch/arm64/include/asm/acpi.h | 3 +++ > drivers/perf/arm_pmu_acpi.c | 35 +++++++++++++++++++++++++++++++++++ > include/linux/perf/arm_pmu.h | 1 + > 3 files changed, 39 insertions(+) > > diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h > index bd68e1b7f29f3..4d537d56eb847 100644 > --- a/arch/arm64/include/asm/acpi.h > +++ b/arch/arm64/include/asm/acpi.h > @@ -42,6 +42,9 @@ > #define ACPI_MADT_GICC_SPE (offsetof(struct acpi_madt_generic_interrupt, \ > spe_interrupt) + sizeof(u16)) > > +#define ACPI_MADT_GICC_TRBE (offsetof(struct acpi_madt_generic_interrupt, \ > + trbe_interrupt) + sizeof(u16)) > + > /* Basic configuration for ACPI */ > #ifdef CONFIG_ACPI > pgprot_t __acpi_get_mem_attribute(phys_addr_t addr); > diff --git a/drivers/perf/arm_pmu_acpi.c b/drivers/perf/arm_pmu_acpi.c > index 90815ad762ebc..8baeeb369118e 100644 > --- a/drivers/perf/arm_pmu_acpi.c > +++ b/drivers/perf/arm_pmu_acpi.c > @@ -139,6 +139,40 @@ static inline void arm_spe_acpi_register_device(void) > } > #endif /* CONFIG_ARM_SPE_PMU */ > > +#if IS_ENABLED(CONFIG_CORESIGHT_TRBE) > +static struct resource trbe_resources[] = { > + { > + /* irq */ > + .flags = IORESOURCE_IRQ, > + } > +}; > + > +static struct platform_device trbe_dev = { > + .name = ARMV8_TRBE_PDEV_NAME, > + .id = -1, > + .resource = trbe_resources, > + .num_resources = ARRAY_SIZE(trbe_resources) > +}; > + > +static u16 arm_trbe_parse_gsi(struct acpi_madt_generic_interrupt *gicc) > +{ > + return gicc->trbe_interrupt; > +} > + > +static void arm_trbe_acpi_register_device(void) > +{ > + int ret = arm_acpi_register_pmu_device(&trbe_dev, ACPI_MADT_GICC_TRBE, > + arm_trbe_parse_gsi); > + if (ret) > + pr_warn("ACPI: TRBE: Unable to register device\n"); > +} > +#else > +static inline void arm_trbe_acpi_register_device(void) > +{ > + > +} > +#endif /* CONFIG_CORESIGHT_TRBE */ > + > static int arm_pmu_acpi_parse_irqs(void) > { > int irq, cpu, irq_cpu, err; > @@ -374,6 +408,7 @@ static int arm_pmu_acpi_init(void) > return 0; > > arm_spe_acpi_register_device(); > + arm_trbe_acpi_register_device(); > > return 0; > } > diff --git a/include/linux/perf/arm_pmu.h b/include/linux/perf/arm_pmu.h > index a0801f68762bf..143fbc10ecfe0 100644 > --- a/include/linux/perf/arm_pmu.h > +++ b/include/linux/perf/arm_pmu.h > @@ -187,5 +187,6 @@ void armpmu_free_irq(int irq, int cpu); > #endif /* CONFIG_ARM_PMU */ > > #define ARMV8_SPE_PDEV_NAME "arm,spe-v1" > +#define ARMV8_TRBE_PDEV_NAME "arm,trbe" > > #endif /* __ARM_PMU_H__ */ _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE 2023-10-16 16:05 ` Thorsten Leemhuis @ 2023-10-16 16:52 ` Greg Kroah-Hartman 2023-10-17 6:06 ` Thorsten Leemhuis 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2023-10-16 16:52 UTC (permalink / raw) To: Thorsten Leemhuis Cc: stable, patches, Catalin Marinas, Will Deacon, Mark Rutland, linux-arm-kernel, linux-kernel, Anshuman Khandual, Sasha Levin On Mon, Oct 16, 2023 at 06:05:53PM +0200, Thorsten Leemhuis wrote: > On 16.10.23 10:39, Greg Kroah-Hartman wrote: > > 6.5-stable review patch. If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Anshuman Khandual <anshuman.khandual@arm.com> > > > > [ Upstream commit 1aa3d0274a4aac338ee45a3dfc3b17c944bcc2bc ] > > > > ACPI TRBE does not have a HID for identification which could create and add > > a platform device into the platform bus. Also without a platform device, it > > cannot be probed and bound to a platform driver. > > > > This creates a dummy platform device for TRBE after ascertaining that ACPI > > provides required interrupts uniformly across all cpus on the system. This > > device gets created inside drivers/perf/arm_pmu_acpi.c to accommodate TRBE > > being built as a module. > > > > Cc: Catalin Marinas <catalin.marinas@arm.com> > > Cc: Will Deacon <will@kernel.org> > > Cc: Mark Rutland <mark.rutland@arm.com> > > Cc: linux-arm-kernel@lists.infradead.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com> > > Link: https://lore.kernel.org/r/20230817055405.249630-3-anshuman.khandual@arm.com > > Signed-off-by: Will Deacon <will@kernel.org> > > Stable-dep-of: 4785aa802853 ("cpuidle, ACPI: Evaluate LPI arch_flags for broadcast timer") > > Signed-off-by: Sasha Levin <sashal@kernel.org> > > Got a compiler error when using Fedora's stable kernel config and > compiling 6.5.8-rc1 for ARM64; not totally sure, but from the error msg > I suspect it's caused by the change quoted above. > > """ > + /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g > -grecord-gcc-switches -pipe -Wall -Werror=format-security > -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS > -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong > -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 > -mbranch-protection=standard -fasynchronous-unwind-tables > -fstack-clash-protection' 'HOSTLDFLAGS=-Wl,-z,relro -Wl,--as-needed > -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld > -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--build-id=sha1 > -specs=/usr/lib/rpm/redhat/redhat-package-notes' ARCH=arm64 'KCFLAGS= ' > WITH_GCOV=0 -j4 vmlinuz.efi > drivers/perf/arm_pmu_acpi.c: In function 'arm_trbe_acpi_register_device': > drivers/perf/arm_pmu_acpi.c:164:19: error: implicit declaration of > function 'arm_acpi_register_pmu_device'; did you mean > 'arm_spe_acpi_register_device'? [-Werror=implicit-function-declaration] > 164 | int ret = arm_acpi_register_pmu_device(&trbe_dev, > ACPI_MADT_GICC_TRBE, > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ > | arm_spe_acpi_register_device > cc1: some warnings being treated as errors > make[4]: *** [scripts/Makefile.build:243: drivers/perf/arm_pmu_acpi.o] > Error 1 > make[4]: *** Waiting for unfinished jobs.... > make[3]: *** [scripts/Makefile.build:480: drivers/perf] Error 2 > make[3]: *** Waiting for unfinished jobs.... > make[2]: *** [scripts/Makefile.build:480: drivers] Error 2 > make[1]: *** > [/builddir/build/BUILD/kernel-6.5.8-rc1/linux-6.5.8-0.rc1.150.vanilla.fc37.aarch64/Makefile:2036: > .] Error 2 > make: *** [Makefile:236: __sub-make] Error 2 > """ > > Full build log: > https://copr-be.cloud.fedoraproject.org/results/@kernel-vanilla/stable-rc/fedora-37-aarch64/06529454-stablerc-stablerc-releases/builder-live.log.gz Thanks, I'll drop the offending commit and push out a -rc2. greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE 2023-10-16 16:52 ` Greg Kroah-Hartman @ 2023-10-17 6:06 ` Thorsten Leemhuis 0 siblings, 0 replies; 4+ messages in thread From: Thorsten Leemhuis @ 2023-10-17 6:06 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: stable, patches, Catalin Marinas, Will Deacon, Mark Rutland, linux-arm-kernel, linux-kernel, Anshuman Khandual, Sasha Levin On 16.10.23 18:52, Greg Kroah-Hartman wrote: > On Mon, Oct 16, 2023 at 06:05:53PM +0200, Thorsten Leemhuis wrote: >> On 16.10.23 10:39, Greg Kroah-Hartman wrote: >>> 6.5-stable review patch. If anyone has any objections, please let me know. >>> >>> ------------------ >>> >>> From: Anshuman Khandual <anshuman.khandual@arm.com> >>> >>> [ Upstream commit 1aa3d0274a4aac338ee45a3dfc3b17c944bcc2bc ] >>> >>> ACPI TRBE does not have a HID for identification which could create and add >>> a platform device into the platform bus. Also without a platform device, it >>> cannot be probed and bound to a platform driver. >> [...] >> Got a compiler error when using Fedora's stable kernel config and >> compiling 6.5.8-rc1 for ARM64; not totally sure, but from the error msg >> I suspect it's caused by the change quoted above. > [...] > Thanks, I'll drop the offending commit and push out a -rc2. Thx, that made things work! Ciao, Thorsten _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-10-17 6:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20231016084015.400031271@linuxfoundation.org>
2023-10-16 8:39 ` [PATCH 6.5 006/191] arm_pmu: acpi: Add a representative platform device for TRBE Greg Kroah-Hartman
2023-10-16 16:05 ` Thorsten Leemhuis
2023-10-16 16:52 ` Greg Kroah-Hartman
2023-10-17 6:06 ` Thorsten Leemhuis
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).