Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750813AbaKUReq (ORCPT ); Fri, 21 Nov 2014 12:34:46 -0500 Received: from cdptpa-outbound-snat.email.rr.com ([107.14.166.229]:55008 "EHLO cdptpa-oedge-vip.email.rr.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750699AbaKURep (ORCPT ); Fri, 21 Nov 2014 12:34:45 -0500 Date: Fri, 21 Nov 2014 12:34:34 -0500 From: Steven Rostedt To: Linus Torvalds Cc: Andy Lutomirski , Tejun Heo , "linux-kernel@vger.kernel.org" , Thomas Gleixner , Arnaldo Carvalho de Melo , Peter Zijlstra , Frederic Weisbecker , Don Zickus , Dave Jones , "the arch/x86 maintainers" Subject: Re: frequent lockups in 3.18rc4 Message-ID: <20141121123434.279c5613@gandalf.local.home> In-Reply-To: References: <20141120221122.GA25393@htj.dyndns.org> <20141120230514.GB25393@htj.dyndns.org> <20141120233920.GC25393@htj.dyndns.org> <20141121162742.GB15461@htj.dyndns.org> <20141121170805.GD30603@home.goodmis.org> X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.25; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-RR-Connecting-IP: 107.14.168.142:25 X-Cloudmark-Score: 0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 21 Nov 2014 09:19:02 -0800 Linus Torvalds wrote: > On Fri, Nov 21, 2014 at 9:08 AM, Steven Rostedt wrote: > > > > Actually, in_nmi() is now safe for vmalloc faults. In fact, it handles the > > clobbering of the cr2 register just fine. > > That's not what I object to and find incorrect wrt NMI. I was commenting about the WARN_ON() itself. > > Compare the simple and correct 32-bit code to the complex and > incorrect 64-bit code. > > In particular, look at how the 32-bit code relies *entirely* on hardware state. > > Then look at where the 64-bit code does not. I see. You have issues with the use of current->active_mm instead of just doing a read_cr3() (and I'm sure other things). Doing a series of git blame, 64 bit has been like that since 2005 (start of git). Looks to me we have more work to do with the merging of 64 bit and 32 bit. Perhaps 64 bit can become more like 32 bit. -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/