From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S937007AbdIZIei (ORCPT ); Tue, 26 Sep 2017 04:34:38 -0400 Received: from terminus.zytor.com ([65.50.211.136]:58523 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934122AbdIZIed (ORCPT ); Tue, 26 Sep 2017 04:34:33 -0400 Date: Tue, 26 Sep 2017 01:27:48 -0700 From: tip-bot for Ingo Molnar Message-ID: Cc: oleg@redhat.com, hpa@zytor.com, yu-cheng.yu@intel.com, riel@redhat.com, linux-kernel@vger.kernel.org, dave.hansen@linux.intel.com, torvalds@linux-foundation.org, luto@kernel.org, mingo@kernel.org, fenghua.yu@intel.com, peterz@infradead.org, tglx@linutronix.de, akpm@linux-foundation.org, ebiggers3@gmail.com, luto@amacapital.net, bp@alien8.de Reply-To: peterz@infradead.org, tglx@linutronix.de, luto@amacapital.net, ebiggers3@gmail.com, akpm@linux-foundation.org, bp@alien8.de, luto@kernel.org, torvalds@linux-foundation.org, dave.hansen@linux.intel.com, mingo@kernel.org, fenghua.yu@intel.com, linux-kernel@vger.kernel.org, riel@redhat.com, yu-cheng.yu@intel.com, hpa@zytor.com, oleg@redhat.com In-Reply-To: <20170923130016.21448-17-mingo@kernel.org> References: <20170923130016.21448-17-mingo@kernel.org> To: linux-tip-commits@vger.kernel.org Subject: [tip:x86/fpu] x86/fpu: Make the fpu state change in fpu__clear() scheduler-atomic Git-Commit-ID: a10b6a16cdad88170f546d008c77453cddf918e6 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: a10b6a16cdad88170f546d008c77453cddf918e6 Gitweb: http://git.kernel.org/tip/a10b6a16cdad88170f546d008c77453cddf918e6 Author: Ingo Molnar AuthorDate: Sat, 23 Sep 2017 14:59:59 +0200 Committer: Ingo Molnar CommitDate: Sun, 24 Sep 2017 13:04:33 +0200 x86/fpu: Make the fpu state change in fpu__clear() scheduler-atomic Do this temporarily only, to make it easier to change the FPU state machine, in particular this change couples the fpu->fpregs_active and fpu->fpstate_active states: they are only set/cleared together (as far as the scheduler sees them). This will be removed by later patches. Cc: Andrew Morton Cc: Andy Lutomirski Cc: Andy Lutomirski Cc: Borislav Petkov Cc: Dave Hansen Cc: Eric Biggers Cc: Fenghua Yu Cc: Linus Torvalds Cc: Oleg Nesterov Cc: Peter Zijlstra Cc: Rik van Riel Cc: Thomas Gleixner Cc: Yu-cheng Yu Link: http://lkml.kernel.org/r/20170923130016.21448-17-mingo@kernel.org Signed-off-by: Ingo Molnar --- arch/x86/kernel/fpu/core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/x86/kernel/fpu/core.c b/arch/x86/kernel/fpu/core.c index bad5724..b7dc383 100644 --- a/arch/x86/kernel/fpu/core.c +++ b/arch/x86/kernel/fpu/core.c @@ -462,9 +462,11 @@ void fpu__clear(struct fpu *fpu) * Make sure fpstate is cleared and initialized. */ if (static_cpu_has(X86_FEATURE_FPU)) { + preempt_disable(); fpu__activate_curr(fpu); user_fpu_begin(); copy_init_fpstate_to_fpregs(); + preempt_enable(); } }