Linux ATA/IDE development
 help / color / mirror / Atom feed
* [PATCH] ata: libata-core: Add NOLPM quirk for PNY CS900 1TB SSD
@ 2026-06-20  2:54 Bryam Vargas via B4 Relay
  2026-06-22 11:21 ` Niklas Cassel
  0 siblings, 1 reply; 2+ messages in thread
From: Bryam Vargas via B4 Relay @ 2026-06-20  2:54 UTC (permalink / raw)
  To: Damien Le Moal, Niklas Cassel; +Cc: linux-kernel, linux-ide

From: Bryam Vargas <hexlabsecurity@proton.me>

The PNY CS900 1TB SSD (Phison PS3111-S11, DRAM-less) drops off the bus
after entering Device-Initiated Slumber during idle. With the default
med_power_with_dipm policy the link goes down (SStatus 1 SControl 300)
and does not recover, forcing the filesystem read-only. Forcing
max_performance keeps the link stable across prolonged idle.

Add a NOLPM quirk so link power management is disabled for this drive
specifically, leaving it intact for other devices on the host.

Cc: stable@vger.kernel.org
Signed-off-by: Bryam Vargas <hexlabsecurity@proton.me>
---
Tested on an Intel Alder Lake AHCI host. With the default
med_power_with_dipm the link drops after prolonged idle (SStatus 1
SControl 300); writes then fail and the filesystem is remounted
read-only, e.g.:

  sd 0:0:0:0: [sda] tag#22 CDB: Write(10) 2a 00 74 38 55 68 00 00 28 00
  EXT4-fs warning (device sda1): ext4_end_bio:368: I/O error 10 writing
    to inode 52429136 (starting block 243731117)
  ata1: EH complete
  ata1.00: detaching (SCSI 0:0:0:0)
  Buffer I/O error on device sda1, logical block 243730861
  sd 0:0:0:0: [sda] Synchronize Cache(10) failed: Result:
    hostbyte=DID_BAD_TARGET driverbyte=DRIVER_OK

Forcing max_performance (which this quirk applies per-device) keeps the
link stable across overnight idle. Drive: model "PNY CS900 1TB SSD", firmware
CS900702. The Phison PS3111-S11 controller is shared across the CS900
line, so other capacities/firmwares may need the same; scoped here to
the unit tested.
---
 drivers/ata/libata-core.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index 3d0027ec33c2..e1d05d4c13ba 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4280,6 +4280,9 @@ static const struct ata_dev_quirks_entry __ata_dev_quirks[] = {
 	/* Apacer models with LPM issues */
 	{ "Apacer AS340*",		NULL,	ATA_QUIRK_NOLPM },
 
+	/* PNY CS900 (Phison PS3111-S11, DRAM-less) drops the link on DIPM */
+	{ "PNY CS900 1TB SSD",		NULL,	ATA_QUIRK_NOLPM },
+
 	/* Silicon Motion models with LPM issues */
 	{ "MD619HXCLDE3TC",		"TCVAID", ATA_QUIRK_NOLPM },
 	{ "MD619GXCLDE3TC",		"TCV35D", ATA_QUIRK_NOLPM },

---
base-commit: 4549871118cf616eecdd2d939f78e3b9e1dddc48
change-id: 20260619-b4-disp-494070d2-c1633393b79f

Best regards,
-- 
bryamzxz <hexlabsecurity@proton.me>



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-06-22 11:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-20  2:54 [PATCH] ata: libata-core: Add NOLPM quirk for PNY CS900 1TB SSD Bryam Vargas via B4 Relay
2026-06-22 11:21 ` Niklas Cassel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox