diff mbox series

[v2,21/24] kselftest/arm64: add HWCAP test for FEAT_S1POE

Message ID 20231027180850.1068089-22-joey.gouly@arm.com
State Superseded
Headers show
Series [v2,01/24] arm64/sysreg: add system register POR_EL{0,1} | expand

Commit Message

Joey Gouly Oct. 27, 2023, 6:08 p.m. UTC
Check that when POE is enabled, the POR_EL0 register is accessible.

Signed-off-by: Joey Gouly <joey.gouly@arm.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Shuah Khan <shuah@kernel.org>
---
 tools/testing/selftests/arm64/abi/hwcap.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

Comments

Mark Brown Oct. 30, 2023, 7:18 p.m. UTC | #1
On Fri, Oct 27, 2023 at 07:08:47PM +0100, Joey Gouly wrote:

> +	{
> +		.name = "POE",
> +		.at_hwcap = AT_HWCAP2,
> +		.hwcap_bit = HWCAP2_POE,
> +		.cpuinfo = "poe",
> +		.sigill_fn = poe_sigill,
> +	},

We should set sigill_reliable here - there's a trap for the POR_EL0 so
the test must fail if the hwcap isn't available.
diff mbox series

Patch

diff --git a/tools/testing/selftests/arm64/abi/hwcap.c b/tools/testing/selftests/arm64/abi/hwcap.c
index e3d262831d91..64bb49fe3f5c 100644
--- a/tools/testing/selftests/arm64/abi/hwcap.c
+++ b/tools/testing/selftests/arm64/abi/hwcap.c
@@ -101,6 +101,12 @@  static void pmull_sigill(void)
 	asm volatile(".inst 0x0ee0e000" : : : );
 }
 
+static void poe_sigill(void)
+{
+	/* mrs x0, POR_EL0 */
+	asm volatile("mrs x0, S3_3_C10_C2_4" : : : "x0");
+}
+
 static void rng_sigill(void)
 {
 	asm volatile("mrs x0, S3_3_C2_C4_0" : : : "x0");
@@ -379,6 +385,13 @@  static const struct hwcap_data {
 		.cpuinfo = "pmull",
 		.sigill_fn = pmull_sigill,
 	},
+	{
+		.name = "POE",
+		.at_hwcap = AT_HWCAP2,
+		.hwcap_bit = HWCAP2_POE,
+		.cpuinfo = "poe",
+		.sigill_fn = poe_sigill,
+	},
 	{
 		.name = "RNG",
 		.at_hwcap = AT_HWCAP2,