From patchwork Mon Jan 16 16:04:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Brown X-Patchwork-Id: 13103478 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 186D1C677F1 for ; Mon, 16 Jan 2023 16:16:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References:Message-Id :MIME-Version:Subject:Date:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=IPfRbl3kIJWCFylYUFHfcu4MGVCZo3d/3IX6lSUExjM=; b=oFSRMWx7Ba5XHb bXCFAOVbdZ/fj1yJgkes9Q8DK0KmJFarfDP1VlKzWne6eE5j9ZOYnEH8RvFEQogdLwqBhx4ccRhCN qJNszBl0TYsMXkMRGXFFgK9nPzDEGRr6P4VF5KTtc0oRuyki2xAidgHRUWfeRnxpXi5UzJPQ15ApY dmi4olY2QKFDbfZPAFxyo8Li6haa/DGQddjj7rxHhSkyLbb899T12JRFOiwauZFDzV7HdQwrftMbQ eT8cXkSDUE1+K1x31NSrXzGBjY0qg0kGBMNi9TYbi0y08u2pkoK2VliwYUcbKnFU6AlZiT7E+uabD Kz4VLZ6+X93/439g+LmA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHS8w-00BH2X-9E; Mon, 16 Jan 2023 16:15:31 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHS1O-00BDG8-4I for linux-arm-kernel@lists.infradead.org; Mon, 16 Jan 2023 16:07:45 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 9FD616104E; Mon, 16 Jan 2023 16:07:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B0C5FC433F0; Mon, 16 Jan 2023 16:07:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1673885261; bh=hPgmSCQ/Mf/iJwYZPL6yzUIBxj4vmjVTGW5er+S6amw=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=oypXQJ9z80Th7EvgZi3fR1iFJRuGmb1FTl7VEp25Ysm3IMrdK0nCBmeQmIyEcglQ0 6ncm5ZgEpAvMsBBW8WXrXYtktZ6nST7IHaMP2taAV5k0Unet/5g134XQyLdol6DpS9 gMO7Iz614HqNbJKMpsKzh6/cW6WULQ/QSLinWyNIsKApAnJJAOEKHOHnXgLsy0Lqbb fx9tSl26gp3sWlkK9gGyKgQgHHeMuXjFn/TivSUQxkFQmhn2LC3lCqpn1vbIjBE5Ww j8s0obRNhLuLotgKuj99FGHBBi/Go+13jfIQ4CHpzRZNICqFfI6UfkskQZMRPlQWke ojW79Dw5p6Q/w== From: Mark Brown Date: Mon, 16 Jan 2023 16:04:53 +0000 Subject: [PATCH v4 18/21] kselftest/arm64: Add test coverage for ZT register signal frames MIME-Version: 1.0 Message-Id: <20221208-arm64-sme2-v4-18-f2fa0aef982f@kernel.org> References: <20221208-arm64-sme2-v4-0-f2fa0aef982f@kernel.org> In-Reply-To: <20221208-arm64-sme2-v4-0-f2fa0aef982f@kernel.org> To: Catalin Marinas , Will Deacon , Oleg Nesterov , Marc Zyngier , James Morse , Alexandru Elisei , Suzuki K Poulose , Oliver Upton , Shuah Khan Cc: Alan Hayward , Luis Machado , Szabolcs Nagy , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kvmarm@lists.linux.dev, linux-kselftest@vger.kernel.org, Mark Brown X-Mailer: b4 0.12-dev-77e06 X-Developer-Signature: v=1; a=openpgp-sha256; l=4706; i=broonie@kernel.org; h=from:subject:message-id; bh=hPgmSCQ/Mf/iJwYZPL6yzUIBxj4vmjVTGW5er+S6amw=; b=owEBbQGS/pANAwAKASTWi3JdVIfQAcsmYgBjxXWvIhOPlmKxWfonU3La3t9M/vnOV/Wdz7+vloqu AkZYLbaJATMEAAEKAB0WIQSt5miqZ1cYtZ/in+ok1otyXVSH0AUCY8V1rwAKCRAk1otyXVSH0Or0B/ 4xVybg4FabQSm8MiO1j6osiYEjbVCAbauH3HTO0zltkKAo4vS2RIkybgbG4xe3GjK3AEl2pEHeQohg Le5oMnO1uTl+4cwOSMAGLEavwQjuVsDnTXDu+RFLiCmV9OTSQOMuw8RCeJP16Bo0yCn+MVS1gM7y3S TUqul+MmNlHCHSFSe3NEH16pMpueCW9ORD5OxIoJRLZr37ZnWPqJGdfCii2Sx8bVwPqMMezfJNrLyi UJt+ISp+S6WX6F9H3pz28YkxGbgwsISxGRv2EicDZMhZmN3r/zjn3OyodPkoER7EeWbk87z79C+p7X Ww0cy9q7FDBW/TuGkMDzvpBGB18427 X-Developer-Key: i=broonie@kernel.org; a=openpgp; fpr=3F2568AAC26998F9E813A1C5C3F436CA30F5D8EB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230116_080742_330550_DC805DB7 X-CRM114-Status: GOOD ( 19.39 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org We should have a ZT register frame with an expected size when ZA is enabled and have no ZT frame when ZA is disabled. Since we don't load any data into ZT we expect the data to all be zeros since the architecture guarantees it will be set to 0 as ZA is enabled. Signed-off-by: Mark Brown --- tools/testing/selftests/arm64/signal/.gitignore | 1 + .../selftests/arm64/signal/testcases/zt_no_regs.c | 51 +++++++++++++ .../selftests/arm64/signal/testcases/zt_regs.c | 85 ++++++++++++++++++++++ 3 files changed, 137 insertions(+) diff --git a/tools/testing/selftests/arm64/signal/.gitignore b/tools/testing/selftests/arm64/signal/.gitignore index e8d2b57f73ec..b7fbb65183e8 100644 --- a/tools/testing/selftests/arm64/signal/.gitignore +++ b/tools/testing/selftests/arm64/signal/.gitignore @@ -5,4 +5,5 @@ sme_* ssve_* sve_* za_* +zt_* !*.[ch] diff --git a/tools/testing/selftests/arm64/signal/testcases/zt_no_regs.c b/tools/testing/selftests/arm64/signal/testcases/zt_no_regs.c new file mode 100644 index 000000000000..34f69bcf821e --- /dev/null +++ b/tools/testing/selftests/arm64/signal/testcases/zt_no_regs.c @@ -0,0 +1,51 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2021 ARM Limited + * + * Verify that using an instruction not supported in streaming mode + * traps when in streaming mode. + */ + +#include +#include +#include + +#include "test_signals_utils.h" +#include "testcases.h" + +static union { + ucontext_t uc; + char buf[1024 * 128]; +} context; + +int zt_no_regs_run(struct tdescr *td, siginfo_t *si, ucontext_t *uc) +{ + size_t offset; + struct _aarch64_ctx *head = GET_BUF_RESV_HEAD(context); + + /* + * Get a signal context which should not have a ZT frame and + * registers in it. + */ + if (!get_current_context(td, &context.uc, sizeof(context))) + return 1; + + head = get_header(head, ZT_MAGIC, GET_BUF_RESV_SIZE(context), &offset); + if (head) { + fprintf(stderr, "Got unexpected ZT context\n"); + return 1; + } + + td->pass = 1; + + return 0; +} + +struct tdescr tde = { + .name = "ZT register data not present", + .descr = "Validate that ZT is not present when ZA is disabled", + .feats_required = FEAT_SME2, + .timeout = 3, + .sanity_disabled = true, + .run = zt_no_regs_run, +}; diff --git a/tools/testing/selftests/arm64/signal/testcases/zt_regs.c b/tools/testing/selftests/arm64/signal/testcases/zt_regs.c new file mode 100644 index 000000000000..e1eb4d5c027a --- /dev/null +++ b/tools/testing/selftests/arm64/signal/testcases/zt_regs.c @@ -0,0 +1,85 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2021 ARM Limited + * + * Verify that using an instruction not supported in streaming mode + * traps when in streaming mode. + */ + +#include +#include +#include + +#include "test_signals_utils.h" +#include "testcases.h" + +static union { + ucontext_t uc; + char buf[1024 * 128]; +} context; + +static void enable_za(void) +{ + /* smstart za; real data is TODO */ + asm volatile(".inst 0xd503457f" : : : ); +} + +int zt_regs_run(struct tdescr *td, siginfo_t *si, ucontext_t *uc) +{ + size_t offset; + struct _aarch64_ctx *head = GET_BUF_RESV_HEAD(context); + struct zt_context *zt; + char *zeros; + + /* + * Get a signal context which should have a ZT frame and registers + * in it. + */ + enable_za(); + if (!get_current_context(td, &context.uc, sizeof(context))) + return 1; + + head = get_header(head, ZT_MAGIC, GET_BUF_RESV_SIZE(context), &offset); + if (!head) { + fprintf(stderr, "No ZT context\n"); + return 1; + } + + zt = (struct zt_context *)head; + if (zt->nregs == 0) { + fprintf(stderr, "Got context with no registers\n"); + return 1; + } + + fprintf(stderr, "Got expected size %u for %d registers\n", + head->size, zt->nregs); + + /* We didn't load any data into ZT so it should be all zeros */ + zeros = malloc(ZT_SIG_REGS_SIZE(zt->nregs)); + if (!zeros) { + fprintf(stderr, "Out of memory, nregs=%u\n", zt->nregs); + return 1; + } + memset(zeros, 0, ZT_SIG_REGS_SIZE(zt->nregs)); + + if (memcmp(zeros, (char *)zt + ZT_SIG_REGS_OFFSET, + ZT_SIG_REGS_SIZE(zt->nregs)) != 0) { + fprintf(stderr, "ZT data invalid\n"); + return 1; + } + + free(zeros); + + td->pass = 1; + + return 0; +} + +struct tdescr tde = { + .name = "ZT register data", + .descr = "Validate that ZT is present and has data when ZA is enabled", + .feats_required = FEAT_SME2, + .timeout = 3, + .sanity_disabled = true, + .run = zt_regs_run, +};