linux-kernel - Re: Regression from 7627a0edef54 ("ata: ahci: Drop low power policy board type") on reboot (but not cold boot)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Z9hXRYQw1-fX0_PY@ryzen>
Date: Mon, 17 Mar 2025 18:09:25 +0100
From: Niklas Cassel <cassel@...nel.org>
To: Hans de Goede <hdegoede@...hat.com>
Cc: Eric <eric.4.debian@...batoulnz.fr>,
	Salvatore Bonaccorso <carnil@...ian.org>,
	Mario Limonciello <mario.limonciello@....com>,
	Christoph Hellwig <hch@...radead.org>,
	Mika Westerberg <mika.westerberg@...ux.intel.com>,
	Damien Le Moal <dlemoal@...nel.org>,
	Jian-Hong Pan <jhp@...lessos.org>, regressions@...ts.linux.dev,
	linux-kernel@...r.kernel.org, stable@...r.kernel.org,
	linux-ide@...r.kernel.org,
	Dieter Mummenschanz <dmummenschanz@....de>
Subject: Re: Regression from 7627a0edef54 ("ata: ahci: Drop low power policy
 board type") on reboot (but not cold boot)

Hello Hans,

On Thu, Mar 13, 2025 at 07:47:11PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 13-Mar-25 4:28 PM, Niklas Cassel wrote:
> > Hello Hans,
> > 
> > On Thu, Mar 13, 2025 at 04:13:24PM +0100, Hans de Goede wrote:
> >>>
> >>> Considering that DIPM seems to work fine on the Maxtor drive, I guess your
> >>> initial suggestion of a Samsung only quirk which only disables LPM on ATI
> >>> is the best way?
> >>
> >> I have no objections against going that route, except that I guess this
> >> should then be something like ATA_QUIRK_NO_DIPM_ON_ATI to not loose the
> >> other LPM modes / savings? AFAIK/IIRC there still is quite some powersaving
> >> to be had without DIPM.
> > 
> > I was thinking like your original suggestion, i.e. setting:
> > ATA_QUIRK_NO_LPM_ON_ATI
> > 
> > for all the Samsung devices that currently have:
> > ATA_QUIRK_NO_NCQ_ON_ATI
> > 
> > Considering that this Samsung device only supports DIPM
> > (and not HIPM), I'm guessing the same is true for the other
> > Samsung devices as well.
> 
> Ah I see ...
> 
> > So we might as well just do:
> > ATA_QUIRK_NO_LPM_ON_ATI
> 
> Yes I agree and that will nicely work as a combination of
> ATA_QUIRK_NO_LPM + ATA_QUIRK_NO_NCQ_ON_ATI functionality
> so using tested code-paths in a slightly new way.

I sent a patch that implements your original suggestion here:
https://lore.kernel.org/linux-ide/20250317170348.1748671-2-cassel@kernel.org/

I forgot to add your Suggested-by tag.
If the patch solves Eric's problem, I could add the tag when applying.


Kind regards,
Niklas