[PATCH] drivers/nvme: Add quirks for device 025e:f1ac

ilya guterman posted 1 patch 9 months ago
There is a newer version of this series
drivers/nvme/host/pci.c | 2 ++
1 file changed, 2 insertions(+)
[PATCH] drivers/nvme: Add quirks for device 025e:f1ac
Posted by ilya guterman 9 months ago
From: Ilya Guterman <amfernusus@gmail.com>

This commit adds NVME_QUIRK_NO_DEEPEST_PS for
device [126f:2262], which belongs to device SOLIDIGM P44 Pro SSDPFKKW020X7

The device frequently have trouble exiting the deepest power state (5),
resulting in the entire disk unresponsive.

Verified by setting nvme_core.default_ps_max_latency_us=10000 and observing them behaving normally.
Also by running with the patch couldn't reproduce the issue after multiple wake up from sleeps.
Running without the patch again reprodcued the issue on the first wake from sleep.
---
 drivers/nvme/host/pci.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c
index d49b69565d04..d62fef76cc07 100644
--- a/drivers/nvme/host/pci.c
+++ b/drivers/nvme/host/pci.c
@@ -3734,6 +3734,8 @@ static const struct pci_device_id nvme_id_table[] = {
 		.driver_data = NVME_QUIRK_NO_DEEPEST_PS, },
 	{ PCI_DEVICE(0x1e49, 0x0041),   /* ZHITAI TiPro7000 NVMe SSD */
 		.driver_data = NVME_QUIRK_NO_DEEPEST_PS, },
+	{ PCI_DEVICE(0x025e, 0xf1ac),   /* SOLIDIGM  P44 pro SSDPFKKW020X7  */
+		.driver_data = NVME_QUIRK_NO_DEEPEST_PS, },
 	{ PCI_DEVICE(0xc0a9, 0x540a),   /* Crucial P2 */
 		.driver_data = NVME_QUIRK_BOGUS_NID, },
 	{ PCI_DEVICE(0x1d97, 0x2263), /* Lexar NM610 */
-- 
2.49.0
Re: [PATCH] drivers/nvme: Add quirks for device 025e:f1ac
Posted by Christoph Hellwig 8 months, 4 weeks ago
On Sat, May 10, 2025 at 07:21:30PM +0900, ilya guterman wrote:
> From: Ilya Guterman <amfernusus@gmail.com>
> 
> This commit adds NVME_QUIRK_NO_DEEPEST_PS for
> device [126f:2262], which belongs to device SOLIDIGM P44 Pro SSDPFKKW020X7
> 
> The device frequently have trouble exiting the deepest power state (5),
> resulting in the entire disk unresponsive.

Does this happen in more than one host system?
Re: [PATCH] drivers/nvme: Add quirks for device 025e:f1ac
Posted by Ilya Guterman 8 months, 4 weeks ago
I've only been able to test on my own host system, but I found similar
reports from others online.

Let me quote from the reddit conversation
https://www.reddit.com/r/buildapcsales/comments/1e4tgge/comment/ldha0ye/

> If it’s the same issues as the Solidigm P44 Pro, don’t recommend. Has a couple issues:
> 2. There also appears to be a NVME controller issue where it disconnects or something. Results in a full system crash and the drive unable to be found upon reboot  attempts. Requires full power cycle. Unsure if this affects the P41 Platinum
> > I experienced #2, it's something relating to the power saving state and it being unable to wake up in time. The fix on Linux is to add the boot parameter nvme_core.default_ps_max_latency_us=0. There exists no fix for Windows.

Here’s a report about the drive disconnecting randomly, although it’s
unclear whether it’s related to waking up from sleep.
https://community.solidigm.com/t5/solid-state-drives-nand/p44-pro-nvme-controller-is-down-will-reset/m-p/24348
Re: [PATCH] drivers/nvme: Add quirks for device 025e:f1ac
Posted by Christoph Hellwig 8 months, 4 weeks ago
On Wed, May 14, 2025 at 04:14:11PM +0900, Ilya Guterman wrote:
> I've only been able to test on my own host system, but I found similar
> reports from others online.
> 
> Let me quote from the reddit conversation
> https://www.reddit.com/r/buildapcsales/comments/1e4tgge/comment/ldha0ye/

Thanks!  I'll also need your Signed-off-by: tag to apply this.

Sorry for only noticing now.
Re: [PATCH] drivers/nvme: Add quirks for device 025e:f1ac
Posted by Ilya Guterman 8 months, 4 weeks ago
my bad, I was sure I added the Signed-off-by tag.
I will submit a new version of the patch.