From mboxrd@z Thu Jan 1 00:00:00 1970 From: Corey Minyard Subject: Re: [PATCH] lockdep: annotate rcu_read_{,un}lock() Date: Fri, 17 Aug 2007 13:48:09 -0500 Message-ID: <46C5ED69.6060604@acm.org> References: <20070815.144628.104052147.davem@davemloft.net> <1187274307.6114.92.camel@twins> <20070816160145.GA16957@linux.vnet.ibm.com> <1187337405.6114.123.camel@twins> <20070817155357.GD8464@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Peter Zijlstra , Ingo Molnar , herbert@gondor.apana.org.au, 123.oleg@gmail.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, David Miller , Daniel Walker , josht@linux.vnet.ibm.com To: paulmck@linux.vnet.ibm.com Return-path: Received: from vms040pub.verizon.net ([206.46.252.40]:63208 "EHLO vms040pub.verizon.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753696AbXHQSsW (ORCPT ); Fri, 17 Aug 2007 14:48:22 -0400 In-reply-to: <20070817155357.GD8464@linux.vnet.ibm.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Paul E. McKenney wrote: > On Fri, Aug 17, 2007 at 09:56:45AM +0200, Peter Zijlstra wrote: > >> On Thu, 2007-08-16 at 09:01 -0700, Paul E. McKenney wrote: >> >>> On Thu, Aug 16, 2007 at 04:25:07PM +0200, Peter Zijlstra wrote: >>> >>>> There seem to be some unbalanced rcu_read_{,un}lock() issues of late, >>>> how about doing something like this: >>>> >>> This will break when rcu_read_lock() and rcu_read_unlock() are invoked >>> from NMI/SMI handlers -- the raw_local_irq_save() in lock_acquire() will >>> not mask NMIs or SMIs. >>> >>> One approach would be to check for being in an NMI/SMI handler, and >>> to avoid calling lock_acquire() and lock_release() in those cases. >>> >> It seems: >> >> #define nmi_enter() do { lockdep_off(); __irq_enter(); } while (0) >> #define nmi_exit() do { __irq_exit(); lockdep_on(); } while (0) >> >> Should make it all work out just fine. (for NMIs at least, /me fully >> ignorant of the workings of SMIs) >> > > Very good point, at least for NMIs on i386 and x86_64. Can't say that I > know much about SMIs myself. Or about whatever equivalents to NMIs and > SMIs might exist on other platforms. :-/ Of course, the other platforms > could be handled by making the RCU lockdep operate only on i386 and x86_64 > if required. > > Corey, any advice on SMI handlers? Is there something like nmi_enter() > and nmi_exit() that allows disabing lockdep? > You will certainly need something like nmi_enter() and nmi_exit() for SMIs, since they can occur at any time like NMIs. As far as anything else, you just have to be extremely careful and remember that it can occur anyplace. But you already know that :). It would be nice if the PowerPC board vendors would tie watchdog pretimeouts and some type of timer into the SMI input. It would make debugging certain problems much easier. And all those Marvell bridge chips have a watchdog pretimeout and I haven't seen any board vendor wire it up :(. -corey