From mboxrd@z Thu Jan 1 00:00:00 1970 From: Suresh Siddha Subject: Re: Kernel oops with 2.6.26, padlock and ipsec: probably problem with fpu state changes Date: Sat, 9 Aug 2008 11:54:13 -0700 Message-ID: <20080809185412.GI13158@linux-os.sc.intel.com> References: <200807171653.59177.wolfgang.walter@stwm.de> <20080808231121.GA13158@linux-os.sc.intel.com> <20080809143727.GA30499@gondor.apana.org.au> <200808091757.32999.wolfgang.walter@stwm.de> <20080809181219.GF13158@linux-os.sc.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Wolfgang Walter , Herbert Xu , "H. Peter Anvin" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Ingo Molnar , "viro@ZenIV.linux.org.uk" , "vegard.nossum@gmail.com" To: Suresh Siddha Return-path: Received: from mga01.intel.com ([192.55.52.88]:45420 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751695AbYHISyO (ORCPT ); Sat, 9 Aug 2008 14:54:14 -0400 Content-Disposition: inline In-Reply-To: <20080809181219.GF13158@linux-os.sc.intel.com> Sender: netdev-owner@vger.kernel.org List-ID: On Sat, Aug 09, 2008 at 11:12:19AM -0700, Suresh Siddha wrote: > In this padlock case with the patch, we may encounter a nested > kernel_fpu_begin() and end() > but this is ok, as the padlock is not actually touching fpu/sse registers. I take this back. kernel_fpu_end() is unconditionally doing stts(). So, nesting of kernel_fpu_begin/end() is not ok.