Message ID | 20090630224612.17771.38476.stgit@localhost.localdomain |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Tue, 30 Jun 2009 15:46:13 -0700 > From: Andre Detsch <adetsch@br.ibm.com> > > PCI drivers that implement the io_error_detected callback > should return PCI_ERS_RESULT_DISCONNECT if the state > passed in is pci_channel_io_perm_failure. This state is > not checked in many of the network drivers. > > The patch fixes the omission in the e1000 driver. > > Based on Mike Mason's similar patch for e1000e. > > Signed-off-by: Andre Detsch <adetsch@br.ibm.com> > CC: Mike Mason <mmlnx@us.ibm.com> > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c index 972e06d..5b8cbdb 100644 --- a/drivers/net/e1000/e1000_main.c +++ b/drivers/net/e1000/e1000_main.c @@ -4823,6 +4823,9 @@ static pci_ers_result_t e1000_io_error_detected(struct pci_dev *pdev, netif_device_detach(netdev); + if (state == pci_channel_io_perm_failure) + return PCI_ERS_RESULT_DISCONNECT; + if (netif_running(netdev)) e1000_down(adapter); pci_disable_device(pdev);