aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Duyck <alexander.h.duyck@intel.com>2010-11-16 22:26:52 -0500
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>2010-11-16 22:26:52 -0500
commit5f5ae6fc86083526088e2c2ca4454e0f44f1e0cb (patch)
tree5bd61adbe1dd33c9dcf1932840958f65893a47ed
parentfc77dc3cc15144bbaf18203e9ef7a3e1beedfc3f (diff)
ixgbe: move ixgbe_clear_interrupt_scheme to before pci_save_state
The main reason for this change is to keep the suspend/resume logic matched up. The clear_interrupt_scheme function will disable MSI-X which will effect the PCIe configuration space. Therefore we will want to do it before we save state to avoid having the interrupt state restored by pci_restore_state, and then trying to re-enable MSI/MSI-X interrupts via ixgbe_setup_interrupt_scheme. Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com> Tested-by: Ross Brattain <ross.b.brattain@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
-rw-r--r--drivers/net/ixgbe/ixgbe_main.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c
index 29523cec2704..cbb3570f920e 100644
--- a/drivers/net/ixgbe/ixgbe_main.c
+++ b/drivers/net/ixgbe/ixgbe_main.c
@@ -5327,6 +5327,8 @@ static int __ixgbe_shutdown(struct pci_dev *pdev, bool *enable_wake)
5327 ixgbe_free_all_rx_resources(adapter); 5327 ixgbe_free_all_rx_resources(adapter);
5328 } 5328 }
5329 5329
5330 ixgbe_clear_interrupt_scheme(adapter);
5331
5330#ifdef CONFIG_PM 5332#ifdef CONFIG_PM
5331 retval = pci_save_state(pdev); 5333 retval = pci_save_state(pdev);
5332 if (retval) 5334 if (retval)
@@ -5360,8 +5362,6 @@ static int __ixgbe_shutdown(struct pci_dev *pdev, bool *enable_wake)
5360 5362
5361 *enable_wake = !!wufc; 5363 *enable_wake = !!wufc;
5362 5364
5363 ixgbe_clear_interrupt_scheme(adapter);
5364
5365 ixgbe_release_hw_control(adapter); 5365 ixgbe_release_hw_control(adapter);
5366 5366
5367 pci_disable_device(pdev); 5367 pci_disable_device(pdev);