http://bugzilla.kernel.org/show_bug.cgi?id=8855
------- Comment #12 from [EMAIL PROTECTED] 2007-09-25 16:34 ------- > ------- Comment #11 from [EMAIL PROTECTED] 2007-09-25 04:17 ------- > Well, first, I don't think that my patch above is the way to go. Still, it > illustrates quite well that there's a difference between device_shutdown() and > device_suspend() that leads to the platform spinning the disk for the second > time in the latter case. > > IMHO, we should try to understand what _exactly_ the difference is. I'm not a > (S)ATA expert, though. If you can prepare a debug patch for me, I'll run it > and try to learn something from that. :-) I think drivers/ata/libata-core.c:ata_pci_device_do_suspend() is what's causing the difference. If you comment out pci_disable_device() and pci_set_power_state() calls in the function, I think you'll see double spindown with device_suspend() too. If you want resume to work properly, you'll need to change pcim_enable_device() call to pci_reenable_device(). Thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
