Messages in this thread |  | | | From | Frans Pop <> | | Subject | Re: [BUG] Suspend tries to suspend devices already in D3 | | Date | Sat, 14 Feb 2009 01:48:32 +0100 |
| |
On Wednesday 04 February 2009, Rafael J. Wysocki wrote: > On Wednesday 04 February 2009, Frans Pop wrote: > > On Wednesday 04 February 2009, Rafael J. Wysocki wrote: > > > On Tuesday 03 February 2009, Frans Pop wrote: > > > > On Tuesday 03 February 2009, Frans Pop wrote: > > > > > Looking at iwl-agn.c the cause could be that iwl_pci_suspend() > > > > > depends on iwl_mac_stop() to call pci_save_state(). I added > > > > > some debug statements in iwl_pci_suspend() that showed > > > > > priv->is_open is false, so iwl_mac_stop does indeed *not* get > > > > > called when the interface is down. > > > > > > > > However, iwl_mac_stop() had been called when I ran ifdown just > > > > before suspending the system: > > > > > > The problem that the warning can be triggered by a device being in > > > a low power state already before suspend should be fixed by the PCI > > > PM fixes I've just posted. > > > > I'm afraid I still get the warning with your latest patches on top of > > v2.6.29-rc3-495-gdcf6a79. > > If you're still getting the warning with these patches, then yes, the > problem appears to be in iwl-agn.c .
FYI, this looks to be fixed in -rc5 by c4e061ace755 + 89c581b3fb29.
Thanks, FJP
|  |