From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oliver Hartkopp Subject: Re: mac80211: NOHZ: local_softirq_pending 08 Date: Wed, 30 Sep 2009 19:51:12 +0200 Message-ID: <4AC39A90.6060602@hartkopp.net> References: <200909111648.50902.mb@bu3sch.de> <87ocosqykb.fsf@purkki.valot.fi> <1254322466.3959.5.camel@johannes.local> <200909301710.31082.mb@bu3sch.de> <1254324077.3959.7.camel@johannes.local> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Michael Buesch , Kalle Valo , "John W. Linville" , linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Johannes Berg Return-path: In-Reply-To: <1254324077.3959.7.camel-YfaajirXv2244ywRPIzf9A@public.gmane.org> Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org Johannes Berg wrote: > On Wed, 2009-09-30 at 17:10 +0200, Michael Buesch wrote: >> On Wednesday 30 September 2009 16:54:26 Johannes Berg wrote: >>> On Wed, 2009-09-30 at 17:47 +0300, Kalle Valo wrote: >>> >>>> I agree with Michael. The bug is real and I have verified that >>>> Michael's patch fixes the issue. Better to apply the patch now, it's >>>> trivial to change the implementation if/when the network stack has >>>> support for this. >>> FWIW, I think in mac80211 the in_interrupt() check can never return true >>> since we postpone all RX to the tasklet. But the tasklet seems to be ok >>> -- so should it really be in_interrupt()? >> I think a tasklet is also in_interrupt(), because it's a softirq. > > Ah, yes, indeed, in_interrupt() vs. in_irq(). > Oops! I missed that for my previous patch i added for two occurrences in the CAN sources. I'm currently compiling the patch for netif_rx_ti() and will post it in some minutes (for CAN and mac80211) when it runs without probs. Regards, Oliver -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html