Hi, On Tuesday 17 August 2010, 13:29:05 Tejun Heo wrote: > Hello, > > On 08/17/2010 01:19 PM, Rafael J. Wysocki wrote: > > Well, I wonder what the real reason for doing the link power management > > thing at this particular point in the suspend code path is. It just seems to > > disable the link power management, but then the controller is put into a > > low-power state and is reset from scratch during resume, so I'm not quite > > sure how skipping that code could possibly lead to any problems. > > > Perhaps we could move the link PM manipulation to the prepare stage > > of suspend? > > Yeah, one possibility is that the devices misbehave if they receive > LPM commands while suspended. Does commenting out sd_suspend resolve > the issue too? If you want me to test anything... let me know. Since I do not know much about the ATA code, I do not know what to change where. (A simple grep for sd_suspend in drivers/ata didn't turn up anything.) Thanks, Stephan -- Stephan Diestelhorst, AMD Operating System Research Center stephan.diestelhorst@xxxxxxx, Tel. +49 (0)351 448 356 719 Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach General Managers: Alberto Bozzo, Andrew Bowd Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632 -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html