PCI PM: Do not disable and enable bridges during suspend-resume
authorRafael J. Wysocki <rjw@sisk.pl>
Wed, 4 Feb 2009 01:01:15 +0000 (02:01 +0100)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Thu, 5 Feb 2009 01:21:26 +0000 (17:21 -0800)
It is a mistake to disable and enable PCI bridges and PCI Express
ports during suspend-resume, at least at the time when it is
currently done.  Disabling them may lead to problems with accessing
devices behind them and they should be automatically enabled when
their standard config spaces are restored.  Fix this by not attempting
to disable bridges during suspend and enable them during resume.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/pci-driver.c

index fdb6a697e05abdf5f7998c25199d6b8dddf7a6da..ac6c9e493f4cb051eb5a9d73b4bf0e861a51dde3 100644 (file)
@@ -434,16 +434,18 @@ static int pci_pm_default_resume(struct pci_dev *pci_dev)
 {
        pci_fixup_device(pci_fixup_resume, pci_dev);
 
-       if (!pci_is_bridge(pci_dev))
-               pci_enable_wake(pci_dev, PCI_D0, false);
+       if (pci_is_bridge(pci_dev))
+               return 0;
 
+       pci_enable_wake(pci_dev, PCI_D0, false);
        return pci_pm_reenable_device(pci_dev);
 }
 
 static void pci_pm_default_suspend_generic(struct pci_dev *pci_dev)
 {
-       /* If device is enabled at this point, disable it */
-       pci_disable_enabled_device(pci_dev);
+       /* If a non-bridge device is enabled at this point, disable it */
+       if (!pci_is_bridge(pci_dev))
+               pci_disable_enabled_device(pci_dev);
        /*
         * Save state with interrupts enabled, because in principle the bus the
         * device is on may be put into a low power state after this code runs.