From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2153 invoked by alias); 12 Jan 2017 11:36:56 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 1104 invoked by uid 89); 12 Jan 2017 11:36:48 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-5.1 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:1587, *next, 2857, Hx-spam-relays-external:ESMTPA X-Spam-User: qpsmtpd, 2 recipients X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 12 Jan 2017 11:36:38 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C9C8D1596; Thu, 12 Jan 2017 03:26:25 -0800 (PST) Received: from e103592.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.72.51.249]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id D07C53F24D; Thu, 12 Jan 2017 03:26:23 -0800 (PST) From: Dave Martin To: linux-arm-kernel@lists.infradead.org Cc: Ard Biesheuvel , Marc Zyngier , Alan Hayward , Christoffer Dall , linux-arch@vger.kernel.org, libc-alpha@sourceware.org, Florian Weimer , Joseph Myers , Szabolcs Nagy , Torvald Riegel , gdb@sourceware.org, Yao Qi Subject: [RFC PATCH 04/10] arm64/sve: Factor out clearing of tasks' SVE regs Date: Thu, 12 Jan 2017 11:36:00 -0000 Message-Id: <1484220369-23970-5-git-send-email-Dave.Martin@arm.com> In-Reply-To: <1484220369-23970-1-git-send-email-Dave.Martin@arm.com> References: <1484220369-23970-1-git-send-email-Dave.Martin@arm.com> X-SW-Source: 2017-01/txt/msg00013.txt.bz2 The patch factors out the code that clears a task's SVE regs on exec(), so that we can reuse it in subsequent patches. Signed-off-by: Dave Martin --- arch/arm64/kernel/fpsimd.c | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/arch/arm64/kernel/fpsimd.c b/arch/arm64/kernel/fpsimd.c index ab2bb62..54d7ed0 100644 --- a/arch/arm64/kernel/fpsimd.c +++ b/arch/arm64/kernel/fpsimd.c @@ -260,6 +260,19 @@ void fpsimd_thread_switch(struct task_struct *next) } } +static void clear_sve_regs(struct task_struct *task) +{ + BUG_ON(task == current && preemptible()); + + BUG_ON((char *)__task_sve_state(task) < (char *)task); + BUG_ON(arch_task_struct_size < + ((char *)__task_sve_state(task) - (char *)task)); + + memset(__task_sve_state(task), 0, + arch_task_struct_size - + ((char *)__task_sve_state(task) - (char *)task)); +} + void fpsimd_flush_thread(void) { if (!system_supports_fpsimd()) @@ -272,13 +285,7 @@ void fpsimd_flush_thread(void) memset(¤t->thread.fpsimd_state, 0, sizeof(struct fpsimd_state)); if (IS_ENABLED(CONFIG_ARM64_SVE) && (elf_hwcap & HWCAP_SVE)) { - BUG_ON((char *)__task_sve_state(current) < (char *)current); - BUG_ON(arch_task_struct_size < - ((char *)__task_sve_state(current) - (char *)current)); - - memset(__task_sve_state(current), 0, - arch_task_struct_size - - ((char *)__task_sve_state(current) - (char *)current)); + clear_sve_regs(current); /* * User tasks must have a valid vector length set, but tasks -- 2.1.4