From mboxrd@z Thu Jan 1 00:00:00 1970 From: "David S. Miller" Subject: Re: tun.c patch to fix "smp_processor_id() in preemptible code" Date: Tue, 19 Oct 2004 17:44:32 -0700 Sender: netdev-bounce@oss.sgi.com Message-ID: <20041019174432.41fcdc17.davem@davemloft.net> References: <1098222676.23367.18.camel@krustophenia.net> <20041019215401.GA16427@gondor.apana.org.au> <1098223857.23367.35.camel@krustophenia.net> <20041019153308.488d34c1.davem@davemloft.net> <1098225729.23628.2.camel@krustophenia.net> <20041019154249.6afcaaad.davem@davemloft.net> <1098226288.23628.6.camel@krustophenia.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: herbert@gondor.apana.org.au, vda@port.imtp.ilyichevsk.odessa.ua, netdev@oss.sgi.com, linux-kernel@vger.kernel.org, maxk@qualcomm.com, irda-users@lists.sourceforge.net Return-path: To: Lee Revell In-Reply-To: <1098226288.23628.6.camel@krustophenia.net> Errors-to: netdev-bounce@oss.sgi.com List-Id: netdev.vger.kernel.org On Tue, 19 Oct 2004 18:51:28 -0400 Lee Revell wrote: > OK, thanks for clarifying. The correct patch is therefore: ... > static inline int netif_rx_ni(struct sk_buff *skb) > { > + preempt_disable(); > int err = netif_rx(skb); You need to put statements after local function variable declarations.