[5.4,44/65] nvme-pci: avoid the deepest sleep state on Kingston A2000 SSDs

From: Thorsten Leemhuis <linux@leemhuis.info>

From: Thorsten Leemhuis <linux@leemhuis.info>

commit 538e4a8c571efdf131834431e0c14808bcfb1004 upstream.

Some Kingston A2000 NVMe SSDs sooner or later get confused and stop
working when they use the deepest APST sleep while running Linux. The
system then crashes and one has to cold boot it to get the SSD working
again.

Kingston seems to known about this since at least mid-September 2020:
https://bbs.archlinux.org/viewtopic.php?pid=1926994#p1926994

Someone working for a German company representing Kingston to the German
press confirmed to me Kingston engineering is aware of the issue and
investigating; the person stated that to their current knowledge only
the deepest APST sleep state causes trouble. Therefore, make Linux avoid
it for now by applying the NVME_QUIRK_NO_DEEPEST_PS to this SSD.

I have two such SSDs, but it seems the problem doesn't occur with them.
I hence couldn't verify if this patch really fixes the problem, but all
the data in front of me suggests it should.

This patch can easily be reverted or improved upon if a better solution
surfaces.

FWIW, there are many reports about the issue scattered around the web;
most of the users disabled APST completely to make things work, some
just made Linux avoid the deepest sleep state:

https://bugzilla.kernel.org/show_bug.cgi?id=195039#c65
https://bugzilla.kernel.org/show_bug.cgi?id=195039#c73
https://bugzilla.kernel.org/show_bug.cgi?id=195039#c74
https://bugzilla.kernel.org/show_bug.cgi?id=195039#c78
https://bugzilla.kernel.org/show_bug.cgi?id=195039#c79
https://bugzilla.kernel.org/show_bug.cgi?id=195039#c80
https://askubuntu.com/questions/1222049/nvmekingston-a2000-sometimes-stops-giving-response-in-ubuntu-18-04dell-inspir
https://community.acer.com/en/discussion/604326/m-2-nvme-ssd-aspire-517-51g-issue-compatibility-kingston-a2000-linux-ubuntu

For the record, some data from 'nvme id-ctrl /dev/nvme0'

NVME Identify Controller:
vid       : 0x2646
ssvid     : 0x2646
mn        : KINGSTON SA2000M81000G
fr        : S5Z42105
[...]
ps    0 : mp:9.00W operational enlat:0 exlat:0 rrt:0 rrl:0
          rwt:0 rwl:0 idle_power:- active_power:-
ps    1 : mp:4.60W operational enlat:0 exlat:0 rrt:1 rrl:1
          rwt:1 rwl:1 idle_power:- active_power:-
ps    2 : mp:3.80W operational enlat:0 exlat:0 rrt:2 rrl:2
          rwt:2 rwl:2 idle_power:- active_power:-
ps    3 : mp:0.0450W non-operational enlat:2000 exlat:2000 rrt:3 rrl:3
          rwt:3 rwl:3 idle_power:- active_power:-
ps    4 : mp:0.0040W non-operational enlat:15000 exlat:15000 rrt:4 rrl:4
          rwt:4 rwl:4 idle_power:- active_power:-

Cc: stable@vger.kernel.org # 4.14+
Signed-off-by: Thorsten Leemhuis <linux@leemhuis.info>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/nvme/host/pci.c |    2 ++
 1 file changed, 2 insertions(+)

Message ID	20210208145811.922714357@linuxfoundation.org
State	New
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, Thorsten Leemhuis <linux@leemhuis.info>, Christoph Hellwig <hch@lst.de> Subject: [PATCH 5.4 44/65] nvme-pci: avoid the deepest sleep state on Kingston A2000 SSDs Date: Mon, 8 Feb 2021 16:01:16 +0100 Message-Id: <20210208145811.922714357@linuxfoundation.org> In-Reply-To: <20210208145810.230485165@linuxfoundation.org> References: <20210208145810.230485165@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [5.4,03/65] USB: serial: option: Adding support for Cinterion MV31 [5.4,04/65] arm64: dts: qcom: c630: keep both touchpad devices enabled [5.4,05/65] Input: i8042 - unbreak Pegatron C15B [5.4,10/65] um: virtio: free vu_dev only with the contained struct device [5.4,12/65] arm64: dts: ls1046a: fix dcfg address range [5.4,13/65] igc: set the default return value to -IGC_ERR_NVM in igc_write_nvm_srwr [5.4,14/65] igc: check return value of ret_val in igc_config_fc_after_link_up [5.4,17/65] net: lapb: Copy the skb before sending a packet [5.4,19/65] r8169: fix WoL on shutdown if CONFIG_DEBUG_SHIRQ is set [5.4,24/65] USB: usblp: dont call usb_set_interface if theres a single alt [5.4,27/65] usb: dwc3: fix clock issue during resume in OTG mode [5.4,31/65] usb: host: xhci-plat: add priv quirk for skip PHY initialization [5.4,32/65] ovl: fix dentry leak in ovl_get_redirect [5.4,34/65] fgraph: Initialize tracing_graph_pause at task creation [5.4,35/65] kretprobe: Avoid re-registration of the same kretprobe earlier [5.4,36/65] libnvdimm/dimm: Avoid race between probe and available_slots_show() [5.4,37/65] genirq/msi: Activate Multi-MSI early when MSI_FLAG_ACTIVATE_EARLY is set [5.4,38/65] xhci: fix bounce buffer usage for non-sg list case [5.4,39/65] cifs: report error instead of invalid when revalidating a dentry fails [5.4,40/65] smb3: Fix out-of-bounds bug in SMB2_negotiate() [5.4,43/65] drm/amd/display: Revert "Fix EDID parsing after resume from suspend" [5.4,44/65] nvme-pci: avoid the deepest sleep state on Kingston A2000 SSDs [5.4,45/65] KVM: SVM: Treat SVM as unsupported when running as an SEV guest [5.4,48/65] mm: hugetlbfs: fix cannot migrate the fallocated HugeTLB page [5.4,49/65] mm: hugetlb: fix a race between freeing and dissolving the page [5.4,51/65] mm: hugetlb: remove VM_BUG_ON_PAGE from page_huge_active [5.4,55/65] x86/apic: Add extra serialization for non-serializing MSRs [5.4,57/65] Input: xpad - sync supported devices with fork on GitHub [5.4,58/65] iommu/vt-d: Do not use flush-queue when caching-mode is on [5.4,59/65] md: Set prev_flush_start and flush_bio in an atomic way [5.4,61/65] neighbour: Prevent a dead entry from updating gc_list [5.4,63/65] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

[5.4,44/65] nvme-pci: avoid the deepest sleep state on Kingston A2000 SSDs

Commit Message

Patch