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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <Z9L5p6hTp6MATJ80@ryzen>
Date: Thu, 13 Mar 2025 16:28:39 +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 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.

So we might as well just do:
ATA_QUIRK_NO_LPM_ON_ATI

to disable both HIPM and DIPM
(since only DIPM would have been enabled without this quirk anyway).


> Yes the most severe problems do seem to come from that specific mix,
> although the long list of other ATI controller quirks also shows those
> controllers are somewhat finicky.

Definitely!


Kind regards,
Niklas