Skip to content

Commit 1fae37a

Browse files
shyjumon-nkeithbusch
authored andcommitted
nvme/pci: Add sleep quirk for Samsung and Toshiba drives
The Samsung SSD SM981/PM981 and Toshiba SSD KBG40ZNT256G on the Lenovo C640 platform experience runtime resume issues when the SSDs are kept in sleep/suspend mode for long time. This patch applies the 'Simple Suspend' quirk to these configurations. With this patch, the issue had not been observed in a 1+ day test. Reviewed-by: Jon Derrick <[email protected]> Reviewed-by: Christoph Hellwig <[email protected]> Signed-off-by: Shyjumon N <[email protected]> Signed-off-by: Keith Busch <[email protected]>
1 parent f25372f commit 1fae37a

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

drivers/nvme/host/pci.c

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2747,6 +2747,18 @@ static unsigned long check_vendor_combination_bug(struct pci_dev *pdev)
27472747
(dmi_match(DMI_BOARD_NAME, "PRIME B350M-A") ||
27482748
dmi_match(DMI_BOARD_NAME, "PRIME Z370-A")))
27492749
return NVME_QUIRK_NO_APST;
2750+
} else if ((pdev->vendor == 0x144d && (pdev->device == 0xa801 ||
2751+
pdev->device == 0xa808 || pdev->device == 0xa809)) ||
2752+
(pdev->vendor == 0x1e0f && pdev->device == 0x0001)) {
2753+
/*
2754+
* Forcing to use host managed nvme power settings for
2755+
* lowest idle power with quick resume latency on
2756+
* Samsung and Toshiba SSDs based on suspend behavior
2757+
* on Coffee Lake board for LENOVO C640
2758+
*/
2759+
if ((dmi_match(DMI_BOARD_VENDOR, "LENOVO")) &&
2760+
dmi_match(DMI_BOARD_NAME, "LNVNB161216"))
2761+
return NVME_QUIRK_SIMPLE_SUSPEND;
27502762
}
27512763

27522764
return 0;

0 commit comments

Comments
 (0)