Re: Aw: Re: Re: [PATCH 0/2] Power management fixes

[Damien Le Moal <dlemoal@kernel.org>]

On 1/31/24 16:29, Dieter Mummenschanz wrote:
> Damien,
> so I've applied the patch to 6.8-rc2. Interesting thing is that the behaviour is 
> exactly the same as before (w/o the patch). Besides not 
> honoring CONFIG_SATA_MOBILE_LPM_POLICY=3 after boot my system refuses to 
> transition into lower power states > pc2 after resume even after letting it sit 
> idle for 10 minutes. Transition is only reached after issuing hdparm -Y. So if 
> the patch restores the original behaviour then why did it stop working?!

Interesting... hdparm -Y puts the drive to sleep while the revert puts the drive
in standby state. Sleep is a near complete shutdown of the drive, while standby
is not.

In any case, I think something else is causing this. Probably PCI or ACPI
related changes. So a bisect would be needed to understand this. But this is
going to be very painful to do for this as each test take a while I guess ? How
long do you need to wait to see the system going into low power state (when it
is working that is) ?

Also, which kernel version is the last one you know is OK ?

> dmesg:
> https://pastes.io/1vmmvhvfub
> Regards
> Dieter
> *Gesendet:* Dienstag, 23. Januar 2024 um 12:52 Uhr
> *Von:* "Damien Le Moal" <dlemoal@kernel.org>
> *An:* "Dieter Mummenschanz" <dmummenschanz@web.de>, linux-ide@vger.kernel.org
> *Betreff:* Re: Aw: Re: [PATCH 0/2] Power management fixes
> On 1/23/24 20:40, Dieter Mummenschanz wrote:
>  > Damien,
>  > sorry for getting back to you so late. So is this patch series just a revert or
>  > is it something new? Can I patch and test against 6.8-rc or should I use 6.7?
>  > Anyway I need at least a couple of days since I'm very busy ATM.
> 
> Yes, the second patch is essentially a revert. If you can test with 6.8-rc1 it
> would be great.
> 
> Thanks.
> 
> --
> Damien Le Moal
> Western Digital Research

-- 
Damien Le Moal
Western Digital Research