Re: [PATCH v5 1/7] arm64: add support for the AMU extension v1
From: Catalin Marinas <catalin.marinas@arm.com>
Date: 2020-02-28 10:32:41
Also in:
linux-doc, linux-pm, lkml
Hi Ionela, On Wed, Feb 26, 2020 at 01:29:41PM +0000, Ionela Voinescu wrote:
quoted hunk ↗ jump to hunk
diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index dbc22d684627..49f0c436928f 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt@@ -318,6 +318,15 @@ Format: <a>,<b> See also Documentation/input/joydev/joystick.rst + amu= [ARM64] + Enables or disables detection, enablement and access to + counter registers of the Activity Monitors Unit (AMU). + Format: amu=[0/1/on/off/y/n] + amu=[0/off/n] ensures access to AMU's counter registers + is not attempted. + amu=[1/on/y] (default) enables detection and access to + AMU's counter registers.
Is the only reason for this parameter to be able to disable the feature if the firmware doesn't support it? According to the Kconfig entry, you may see weird behaviour, firmware lock-up. Is the user supposed to try again with amu=0? I'm not particularly fond of adding kernel parameters to work around broken firmware. We have other architecture features (e.g. PtrAuth) that need enabling at EL3 but we don't have such parameters. If it's likely that we hit this issue in practice, I'd rather have the firmware describing the presence of AMU via some DT entry. But I'd rather not bother at all, just get the vendors to update their firmware. If we drop this parameter, patch 1 would need to change. Otherwise the patches look fine. Thanks. -- Catalin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel