| Message ID | 20230202-arm64-kselftest-sve-za-fa64-v1-1-5c5f3dabe441@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 2c4192c0a7f2d628b5c1667577316ee9e7471e20 |
| Headers | show |
| Series | kselftest/arm64: Don't require FA64 for streaming SVE+ZA tests | expand |
On Thu, 02 Feb 2023 17:31:25 +0000, Mark Brown wrote: > During early development a dependedncy was added on having FA64 > available so we could use the full FPSIMD register set in the signal > handler which got copied over into the SSVE+ZA registers test case. > Subsequently the ABI was finialised so the handler is run with streaming > mode disabled meaning this is redundant but the dependency was never > removed, do so now. > > [...] Applied to arm64 (for-next/kselftest), thanks! [1/1] kselftest/arm64: Don't require FA64 for streaming SVE+ZA tests https://git.kernel.org/arm64/c/2c4192c0a7f2
diff --git a/tools/testing/selftests/arm64/signal/testcases/ssve_za_regs.c b/tools/testing/selftests/arm64/signal/testcases/ssve_za_regs.c index 1f62621794d5..9dc5f128bbc0 100644 --- a/tools/testing/selftests/arm64/signal/testcases/ssve_za_regs.c +++ b/tools/testing/selftests/arm64/signal/testcases/ssve_za_regs.c @@ -154,12 +154,7 @@ static int sme_regs(struct tdescr *td, siginfo_t *si, ucontext_t *uc) struct tdescr tde = { .name = "Streaming SVE registers", .descr = "Check that we get the right Streaming SVE registers reported", - /* - * We shouldn't require FA64 but things like memset() used in the - * helpers might use unsupported instructions so for now disable - * the test unless we've got the full instruction set. - */ - .feats_required = FEAT_SME | FEAT_SME_FA64, + .feats_required = FEAT_SME, .timeout = 3, .init = sme_get_vls, .run = sme_regs,
During early development a dependedncy was added on having FA64 available so we could use the full FPSIMD register set in the signal handler which got copied over into the SSVE+ZA registers test case. Subsequently the ABI was finialised so the handler is run with streaming mode disabled meaning this is redundant but the dependency was never removed, do so now. Signed-off-by: Mark Brown <broonie@kernel.org> --- tools/testing/selftests/arm64/signal/testcases/ssve_za_regs.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) --- base-commit: 7294f24db4fa5ebb5a6bde104f08d3345ecee053 change-id: 20230202-arm64-kselftest-sve-za-fa64-9a04f0c49052 Best regards,