Commit a3c69fef authored by Jesse Brandeburg's avatar Jesse Brandeburg Committed by David S. Miller

e1000e: fix close interrupt race

As noticed by Alan Cox, it is possible for e1000e to exit its interrupt
handler or NAPI with interrupts enabled even when the driver is unloading or
being configured administratively down.

fix related to fix for: http://bugzilla.kernel.org/show_bug.cgi?id=12876Signed-off-by: default avatarJesse Brandeburg <jesse.brandeburg@intel.com>
CC: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a72d2b2c
...@@ -1261,7 +1261,8 @@ static irqreturn_t e1000_msix_other(int irq, void *data) ...@@ -1261,7 +1261,8 @@ static irqreturn_t e1000_msix_other(int irq, void *data)
u32 icr = er32(ICR); u32 icr = er32(ICR);
if (!(icr & E1000_ICR_INT_ASSERTED)) { if (!(icr & E1000_ICR_INT_ASSERTED)) {
ew32(IMS, E1000_IMS_OTHER); if (!test_bit(__E1000_DOWN, &adapter->state))
ew32(IMS, E1000_IMS_OTHER);
return IRQ_NONE; return IRQ_NONE;
} }
...@@ -1278,7 +1279,8 @@ static irqreturn_t e1000_msix_other(int irq, void *data) ...@@ -1278,7 +1279,8 @@ static irqreturn_t e1000_msix_other(int irq, void *data)
} }
no_link_interrupt: no_link_interrupt:
ew32(IMS, E1000_IMS_LSC | E1000_IMS_OTHER); if (!test_bit(__E1000_DOWN, &adapter->state))
ew32(IMS, E1000_IMS_LSC | E1000_IMS_OTHER);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -2015,10 +2017,12 @@ static int e1000_clean(struct napi_struct *napi, int budget) ...@@ -2015,10 +2017,12 @@ static int e1000_clean(struct napi_struct *napi, int budget)
if (adapter->itr_setting & 3) if (adapter->itr_setting & 3)
e1000_set_itr(adapter); e1000_set_itr(adapter);
napi_complete(napi); napi_complete(napi);
if (adapter->msix_entries) if (!test_bit(__E1000_DOWN, &adapter->state)) {
ew32(IMS, adapter->rx_ring->ims_val); if (adapter->msix_entries)
else ew32(IMS, adapter->rx_ring->ims_val);
e1000_irq_enable(adapter); else
e1000_irq_enable(adapter);
}
} }
return work_done; return work_done;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment