On Thu, Jul 30, 2009 at 05:37:09PM +0200, Johannes Berg wrote:

On Thu, 2009-07-30 at 10:45 -0400, Bob Copeland wrote:

quoted

On Thu, Jul 30, 2009 at 10:31 AM, Michael Buesch[off-list ref] wrote:

quoted

Ath5k from today's compat-wireless triggers the following WARNing
on hostapd startup.
[ 1669.799870] ------------[ cut here ]------------
[ 1669.799934] WARNING: at kernel/softirq.c:141 local_bh_enable_ip+0x8b/0xb0()

because ath5k calls ieee80211_beacon_get inside a spin_lock_irqsave()
and this is WARN_ON_ONCE(in_irq() || irqs_disabled).

quoted

quoted

[ 1669.800493]  [<ffffffff807e0fef>] _spin_unlock_bh+0xf/0x20
[ 1669.800564]  [<ffffffffa0040b92>] ieee80211_beacon_get+0x2c2/0x2d0 [mac80211]

I guess this is the first victim of spin_lock_bh conversion?

Indeed. I think this means we need to convert the local->sta_lock back
to _irqsave/restore.

John, can you revert the BH patch ("mac80211: do not use irq locks where
not necessary") and I'll send a new one with those changes removed?

OK, will be gone today...

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.
			¡Viva Honduras Libre!