The nic controller's clock on/off irq (CSR_INT_BIT_MAC_CLK_ACTV) is generated when device goes on/off
Unlikely turning on the device, irq by off the device may cause oops because iwl_pci_remove already freed required resources
The irq_tasklet should not be invoked at that condition
$sudo modprobe -r iwl3945
BUG: unable to handle kernel paging request at virtual address f8864008
printing eip: f8b633bd *pdpt = 0000000000003001 *pde = 0000000037c12067 *pte = 0000000000000000
Oops: 0000 [#1] SMP
Modules linked in: netconsole iwl3945 e1000
EIP is at iwl_irq_tasklet+0x82d/0x1010 [iwl3945]
...
Call Trace:
[<c40053ba>] show_trace_log_lvl+0x1a/0x30
[<c4005479>] show_stack_log_lvl+0xa9/0xd0
[<c400556a>] show_registers+0xca/0x1c0
[<c4005776>] die+0x116/0x230
[<c4020cbb>] do_page_fault+0x35b/0x7f0
[<c43eeeda>] error_code+0x72/0x78
[<c4032ceb>] tasklet_action+0x4b/0xc0
[<c4032687>] __do_softirq+0x87/0x100
[<c4032757>] do_softirq+0x57/0x60
...
EIP: [<f8b633bd>] iwl_irq_tasklet+0x82d/0x1010 [iwl3945] SS:ESP 0068:f7c4ff1c
Kernel panic - not syncing: Fatal exception in interrupt
Signed-off-by: Joonwoo Park <joonwpark81@xxxxxxxxx>
---
drivers/net/wireless/iwlwifi/iwl3945-base.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/drivers/net/wireless/iwlwifi/iwl3945-base.c b/drivers/net/wireless/iwlwifi/iwl3945-base.c
index 3986aaf..f95f226 100644
--- a/drivers/net/wireless/iwlwifi/iwl3945-base.c
+++ b/drivers/net/wireless/iwlwifi/iwl3945-base.c
@@ -4867,8 +4867,11 @@ static irqreturn_t iwl_isr(int irq, void *data)
IWL_DEBUG_ISR("ISR inta 0x%08x, enabled 0x%08x, fh 0x%08x\n",
inta, inta_mask, inta_fh);
+ inta &= ~CSR_INT_BIT_MAC_CLK_ACTV;
+
/* iwl_irq_tasklet() will service interrupts and re-enable them */
- tasklet_schedule(&priv->irq_tasklet);
+ if (likely(inta || inta_fh))
+ tasklet_schedule(&priv->irq_tasklet);
unplugged:
spin_unlock(&priv->lock);
--
1.5.3.rc5
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
