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

Salvatore Bonaccorso
2025-03-02T19:28:59Z

On Sun, Mar 02, 2025 at 05:03:48PM +0100, Salvatore Bonaccorso wrote:
> Hi Mario et al,
>
> Eric Degenetais reported in Debian (cf. https://bugs.debian.org/1091696) for
> his report, that after 7627a0edef54 ("ata: ahci: Drop low power policy board
> type") rebooting the system fails (but system boots fine if cold booted).
>
> His report mentions that the SSD is not seen on warm reboots anymore.
>
> Does this ring some bell which might be caused by the above bisected[1] commit?
>
> #regzbot introduced: 7627a0edef54
> #regzbot link: https://bugs.debian.org/1091696
>
> What information to you could be helpful to identify the problem?

Additional information from the reporter: The SSD is:

$ sudo smartctl -i /dev/disk/by-id/ata-Samsung_SSD_870_QVO_2TB_S5RPNF0T419459E
smartctl 7.4 2023-08-01 r5530 [x86_64-linux-6.12.12-amd64] (local build)
Copyright (C) 2002-23, Bruce Allen, Christian Franke, www.smartmontools.org

=== START OF INFORMATION SECTION ===
Model Family: Samsung based SSDs
Device Model: Samsung SSD 870 QVO 2TB
Serial Number: S5RPNF0T419459E
LU WWN Device Id: 5 002538 f4243493c
Firmware Version: SVQ02B6Q
User Capacity: 2 000 398 934 016 bytes [2,00 TB]
Sector Size: 512 bytes logical/physical
Rotation Rate: Solid State Device
Form Factor: 2.5 inches
TRIM Command: Available, deterministic, zeroed
Device is: In smartctl database 7.3/5528
ATA Version is: ACS-4 T13/BSR INCITS 529 revision 5
SATA Version is: SATA 3.3, 6.0 Gb/s (current: 6.0 Gb/s)
Local Time is: Sun Mar 2 18:46:44 2025 CET
SMART support is: Available - device has SMART capability.
SMART support is: Enabled

So this might be the same issue aimed to be addressed by cc77e2ce187d
("ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO
drives"), but which got reverted with a2f925a2f622 ("Revert "ata:
libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives"") as
it introduces other problems.

So I'm adding as well Daniel Baumann into the loop as this seems
related.

FTR, thanks Christian Heusel for the other comments an input!

Regards,
Salvatore