diff -urp a/drivers/net/wireless/ipw2200.c b/drivers/net/wireless/ipw2200.c --- a/drivers/net/wireless/ipw2200.c 2006-04-01 09:47:24.000000000 +0800 +++ b/drivers/net/wireless/ipw2200.c 2006-06-05 11:32:18.000000000 +0800 @@ -542,7 +542,7 @@ static inline void ipw_clear_bit(struct ipw_write32(priv, reg, ipw_read32(priv, reg) & ~mask); } -static inline void ipw_enable_interrupts(struct ipw_priv *priv) +static inline void __ipw_enable_interrupts(struct ipw_priv *priv) { if (priv->status & STATUS_INT_ENABLED) return; @@ -550,7 +550,7 @@ static inline void ipw_enable_interrupts ipw_write32(priv, IPW_INTA_MASK_R, IPW_INTA_MASK_ALL); } -static inline void ipw_disable_interrupts(struct ipw_priv *priv) +static inline void __ipw_disable_interrupts(struct ipw_priv *priv) { if (!(priv->status & STATUS_INT_ENABLED)) return; @@ -558,6 +558,20 @@ static inline void ipw_disable_interrupt ipw_write32(priv, IPW_INTA_MASK_R, ~IPW_INTA_MASK_ALL); } +static inline void ipw_enable_interrupts(struct ipw_priv *priv) +{ + spin_lock_irqsave(&priv->irq_lock, priv->lock_flags); + __ipw_enable_interrupts(priv); + spin_unlock_irqrestore(&priv->irq_lock, priv->lock_flags); +} + +static inline void ipw_disable_interrupts(struct ipw_priv *priv) +{ + spin_lock_irqsave(&priv->irq_lock, priv->lock_flags); + __ipw_disable_interrupts(priv); + spin_unlock_irqrestore(&priv->irq_lock, priv->lock_flags); +} + #ifdef CONFIG_IPW2200_DEBUG static char *ipw_error_desc(u32 val) { @@ -1959,7 +1973,7 @@ static void ipw_irq_tasklet(struct ipw_p unsigned long flags; int rc = 0; - spin_lock_irqsave(&priv->lock, flags); + spin_lock_irqsave(&priv->irq_lock, flags); inta = ipw_read32(priv, IPW_INTA_RW); inta_mask = ipw_read32(priv, IPW_INTA_MASK_R); @@ -1968,6 +1982,10 @@ static void ipw_irq_tasklet(struct ipw_p /* Add any cached INTA values that need to be handled */ inta |= priv->isr_inta; + spin_unlock_irqrestore(&priv->irq_lock, flags); + + spin_lock_irqsave(&priv->lock, flags); + /* handle all the justifications for the interrupt */ if (inta & IPW_INTA_BIT_RX_TRANSFER) { ipw_rx(priv); @@ -2096,10 +2114,10 @@ static void ipw_irq_tasklet(struct ipw_p IPW_ERROR("Unhandled INTA bits 0x%08x\n", inta & ~handled); } + spin_unlock_irqrestore(&priv->lock, flags); + /* enable all interrupts */ ipw_enable_interrupts(priv); - - spin_unlock_irqrestore(&priv->lock, flags); } #define IPW_CMD(x) case IPW_CMD_ ## x : return #x @@ -11058,7 +11076,7 @@ static irqreturn_t ipw_isr(int irq, void if (!priv) return IRQ_NONE; - spin_lock(&priv->lock); + spin_lock(&priv->irq_lock); if (!(priv->status & STATUS_INT_ENABLED)) { /* Shared IRQ */ @@ -11080,7 +11098,7 @@ static irqreturn_t ipw_isr(int irq, void } /* tell the device to stop sending interrupts */ - ipw_disable_interrupts(priv); + __ipw_disable_interrupts(priv); /* ack current interrupts */ inta &= (IPW_INTA_MASK_ALL & inta_mask); @@ -11091,11 +11109,11 @@ static irqreturn_t ipw_isr(int irq, void tasklet_schedule(&priv->irq_tasklet); - spin_unlock(&priv->lock); + spin_unlock(&priv->irq_lock); return IRQ_HANDLED; none: - spin_unlock(&priv->lock); + spin_unlock(&priv->irq_lock); return IRQ_NONE; } @@ -12185,6 +12203,7 @@ static int ipw_pci_probe(struct pci_dev #ifdef CONFIG_IPW2200_DEBUG ipw_debug_level = debug; #endif + spin_lock_init(&priv->irq_lock); spin_lock_init(&priv->lock); for (i = 0; i < IPW_IBSS_MAC_HASH_SIZE; i++) INIT_LIST_HEAD(&priv->ibss_mac_hash[i]); diff -urp a/drivers/net/wireless/ipw2200.h b/drivers/net/wireless/ipw2200.h --- a/drivers/net/wireless/ipw2200.h 2006-04-01 09:47:24.000000000 +0800 +++ b/drivers/net/wireless/ipw2200.h 2006-06-05 11:32:18.000000000 +0800 @@ -1181,6 +1181,8 @@ struct ipw_priv { struct ieee80211_device *ieee; spinlock_t lock; + spinlock_t irq_lock; + unsigned long lock_flags; struct mutex mutex; /* basic pci-network driver stuff */