From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932656AbbHKWid (ORCPT ); Tue, 11 Aug 2015 18:38:33 -0400 Received: from mail-wi0-f177.google.com ([209.85.212.177]:33715 "EHLO mail-wi0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752586AbbHKWib (ORCPT ); Tue, 11 Aug 2015 18:38:31 -0400 Date: Wed, 12 Aug 2015 00:38:28 +0200 From: Frederic Weisbecker To: dvlasenk@redhat.com, riel@redhat.com, bp@alien8.de, peterz@infradead.org, brgerst@gmail.com, vda.linux@googlemail.com, keescook@chromium.org, tglx@linutronix.de, oleg@redhat.com, luto@kernel.org, luto@amacapital.net, torvalds@linux-foundation.org, mingo@kernel.org, hpa@zytor.com, linux-kernel@vger.kernel.org Cc: linux-tip-commits@vger.kernel.org Subject: Re: [tip:x86/asm] x86/asm/entry/64: Migrate error and IRQ exit work to C and remove old assembly code Message-ID: <20150811223827.GB15639@lerouge> References: <60e90901eee611e59e958bfdbbe39969b4f88fe5.1435952415.git.luto@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 07, 2015 at 03:53:29AM -0700, tip-bot for Andy Lutomirski wrote: > Commit-ID: 02bc7768fe447ae305e924b931fa629073a4a1b9 > Gitweb: http://git.kernel.org/tip/02bc7768fe447ae305e924b931fa629073a4a1b9 > Author: Andy Lutomirski > AuthorDate: Fri, 3 Jul 2015 12:44:31 -0700 > Committer: Ingo Molnar > CommitDate: Tue, 7 Jul 2015 10:59:08 +0200 > > x86/asm/entry/64: Migrate error and IRQ exit work to C and remove old assembly code > > Signed-off-by: Andy Lutomirski > Cc: Andy Lutomirski > Cc: Borislav Petkov > Cc: Brian Gerst > Cc: Denys Vlasenko > Cc: Denys Vlasenko > Cc: Frederic Weisbecker > Cc: H. Peter Anvin > Cc: Kees Cook > Cc: Linus Torvalds > Cc: Oleg Nesterov > Cc: Peter Zijlstra > Cc: Rik van Riel > Cc: Thomas Gleixner > Cc: paulmck@linux.vnet.ibm.com > Link: http://lkml.kernel.org/r/60e90901eee611e59e958bfdbbe39969b4f88fe5.1435952415.git.luto@kernel.org > Signed-off-by: Ingo Molnar > --- > arch/x86/entry/entry_64.S | 64 +++++++++++----------------------------- > arch/x86/entry/entry_64_compat.S | 5 ++++ > 2 files changed, 23 insertions(+), 46 deletions(-) > > diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S > index 83eb63d..168ee26 100644 > --- a/arch/x86/entry/entry_64.S > +++ b/arch/x86/entry/entry_64.S > @@ -1088,7 +1055,12 @@ ENTRY(error_entry) > SWAPGS > > .Lerror_entry_from_usermode_after_swapgs: > +#ifdef CONFIG_CONTEXT_TRACKING > + call enter_from_user_mode > +#endif This makes me very nervous as well! It means that instead of using the context tracking save/restore model that we had with exception_enter/exception_exit(), now we rely on the CS register. I don't think we can do that because our "context tracking" is a soft tracking whereas CS is hard tracking and both are not atomically synchronized together. Imagine this situation: we are running in userspace. Context tracking knows it, everything is fine. Now we do a syscall, we enter in kernel entry code but we trigger an exception (DEBUG for example) before we got a chance to call user_exit(), which means that the context tracking code still thinks we are in userspace, so we look at CS from the exception entry code and it says the exception happened in the kernel. Hence we don't call user_exit() before calling the exception handler. There is the bug because the exception handler may use RCU which still thinks we run in userspace. In early context tracking days we have relied on CS. But I changed that because of such issue. The only reliable source for soft context tracking is the soft context tracking itself.