From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Linton Subject: [PATCH v12 0/7] Enable PMUs in ACPI systems Date: Tue, 10 Jan 2017 11:17:45 -0600 Message-ID: <1484068672-15852-1-git-send-email-jeremy.linton@arm.com> Return-path: Received: from foss.arm.com ([217.140.101.70]:59204 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760037AbdAJRRy (ORCPT ); Tue, 10 Jan 2017 12:17:54 -0500 Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: linux-arm-kernel@lists.infradead.org Cc: linux-acpi@vger.kernel.org, will.deacon@arm.com, mark.rutland@arm.com, punit.agrawal@arm.com, steve.capper@arm.com, msalter@redhat.com, mlangsdorf@redhat.com, linux@armlinux.org.uk This patch expands and reworks the patches published by Mark Salter in order to clean up a few of the previous review comments, as well as add support for newer CPUs and big/little configurations. v12: - Rebase to 4.10 - Removed lock in the hotplug path. v11: - Add is_smp() check to read_specific_cpuid() for arch/arm. Update c_show() and various routines in arm_pmu_acpi() to use the macro. - Moved the duplicate "generic" pmu detection code into its own patch and hoist it into arm_pmu_device_probe() so it works for DT based systems as well. v10: - Rebase to 4.9 - Rework the arm_perf_start_cpu changes to support the 4.9 hotplug changes. - Remove the call to acpi_register_gsi() from the cpu online code path. Instead the GSI's are registered during the initcall. This changes the error handling a bit because we now try to clean up the previously registered GSIs in a couple important places. This was also a result of the rebase. - Dropped the MIDR partnumber usage, its no longer necessary to differentiate by only the partnum, so this helps to clarify the code a bit. - Shuffle some code around and rename a few variables. - Added a few comments to hopefully clarify some questions people have previously had about unused MADT entries, skipping processing cores with MIDR=0, etc. v9: - Add/cleanup an additional hotplug patch I've had sitting around. This patch brings the ACPI PMU mostly on par with the DT functionality with respect to having CPUs offline during boot. This should help clarify some of the code structuring. - Cleanup the list of PMU types early if we fail to allocate memory for an additional pmu type. v8: - Rebase to 4.8rc4 - Assorted minor comment/hunk placement/etc tweaks per Punit Agrawal v7: - Rebase to 4.8rc3 - Remove cpu affinity sysfs entry. While providing a CPU mask for ARMv8 PMU's is really helpful in big/little environments, reworking the PMU code to support the cpumask attribute for !arm64 PMUs is out of the scope of this patch set. - Fix CPU miscount problem where an alloc failure followed by successfully allocating the structure can result in under counting the CPUs associated with the PMU. This bug was created in v6 with the conversion to a linked list. - Remove initial platform device creation code by Mark Salter, and re-squash multiple platform device creation code together with helper routines. Other minor tweakage. Jeremy Linton (6): arm64: Rename the common MADT parse routine arm: arm64: Add routine to determine cpuid of other cpus arm: arm64: pmu: Assign platform PMU CPU affinity arm64: pmu: Detect multiple generic PMUs and append counter arm64: pmu: Detect and enable multiple PMUs in an ACPI system arm: pmu: Add PMU definitions for cores not initially online Mark Salter (1): arm64: pmu: Cache PMU interrupt numbers from MADT parse arch/arm/include/asm/cputype.h | 4 + arch/arm/kernel/setup.c | 2 +- arch/arm64/include/asm/cputype.h | 3 + arch/arm64/kernel/perf_event.c | 2 +- arch/arm64/kernel/smp.c | 18 ++- drivers/perf/Kconfig | 4 + drivers/perf/Makefile | 1 + drivers/perf/arm_pmu.c | 103 +++++++++++++-- drivers/perf/arm_pmu_acpi.c | 271 +++++++++++++++++++++++++++++++++++++++ include/linux/perf/arm_pmu.h | 12 ++ 10 files changed, 400 insertions(+), 20 deletions(-) create mode 100644 drivers/perf/arm_pmu_acpi.c -- 2.5.5 From mboxrd@z Thu Jan 1 00:00:00 1970 From: jeremy.linton@arm.com (Jeremy Linton) Date: Tue, 10 Jan 2017 11:17:45 -0600 Subject: [PATCH v12 0/7] Enable PMUs in ACPI systems Message-ID: <1484068672-15852-1-git-send-email-jeremy.linton@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org This patch expands and reworks the patches published by Mark Salter in order to clean up a few of the previous review comments, as well as add support for newer CPUs and big/little configurations. v12: - Rebase to 4.10 - Removed lock in the hotplug path. v11: - Add is_smp() check to read_specific_cpuid() for arch/arm. Update c_show() and various routines in arm_pmu_acpi() to use the macro. - Moved the duplicate "generic" pmu detection code into its own patch and hoist it into arm_pmu_device_probe() so it works for DT based systems as well. v10: - Rebase to 4.9 - Rework the arm_perf_start_cpu changes to support the 4.9 hotplug changes. - Remove the call to acpi_register_gsi() from the cpu online code path. Instead the GSI's are registered during the initcall. This changes the error handling a bit because we now try to clean up the previously registered GSIs in a couple important places. This was also a result of the rebase. - Dropped the MIDR partnumber usage, its no longer necessary to differentiate by only the partnum, so this helps to clarify the code a bit. - Shuffle some code around and rename a few variables. - Added a few comments to hopefully clarify some questions people have previously had about unused MADT entries, skipping processing cores with MIDR=0, etc. v9: - Add/cleanup an additional hotplug patch I've had sitting around. This patch brings the ACPI PMU mostly on par with the DT functionality with respect to having CPUs offline during boot. This should help clarify some of the code structuring. - Cleanup the list of PMU types early if we fail to allocate memory for an additional pmu type. v8: - Rebase to 4.8rc4 - Assorted minor comment/hunk placement/etc tweaks per Punit Agrawal v7: - Rebase to 4.8rc3 - Remove cpu affinity sysfs entry. While providing a CPU mask for ARMv8 PMU's is really helpful in big/little environments, reworking the PMU code to support the cpumask attribute for !arm64 PMUs is out of the scope of this patch set. - Fix CPU miscount problem where an alloc failure followed by successfully allocating the structure can result in under counting the CPUs associated with the PMU. This bug was created in v6 with the conversion to a linked list. - Remove initial platform device creation code by Mark Salter, and re-squash multiple platform device creation code together with helper routines. Other minor tweakage. Jeremy Linton (6): arm64: Rename the common MADT parse routine arm: arm64: Add routine to determine cpuid of other cpus arm: arm64: pmu: Assign platform PMU CPU affinity arm64: pmu: Detect multiple generic PMUs and append counter arm64: pmu: Detect and enable multiple PMUs in an ACPI system arm: pmu: Add PMU definitions for cores not initially online Mark Salter (1): arm64: pmu: Cache PMU interrupt numbers from MADT parse arch/arm/include/asm/cputype.h | 4 + arch/arm/kernel/setup.c | 2 +- arch/arm64/include/asm/cputype.h | 3 + arch/arm64/kernel/perf_event.c | 2 +- arch/arm64/kernel/smp.c | 18 ++- drivers/perf/Kconfig | 4 + drivers/perf/Makefile | 1 + drivers/perf/arm_pmu.c | 103 +++++++++++++-- drivers/perf/arm_pmu_acpi.c | 271 +++++++++++++++++++++++++++++++++++++++ include/linux/perf/arm_pmu.h | 12 ++ 10 files changed, 400 insertions(+), 20 deletions(-) create mode 100644 drivers/perf/arm_pmu_acpi.c -- 2.5.5