Andrew wrote:
> A friend of mine has a brand new LG laptop that has intermittent NVMe
> failures. They mostly happen during a suspend/resume cycle
> (apparently during suspend, not resume). Unlike the earlier
> Dell/Samsung issue, the NVMe device isn't completely gone -- MMIO
> reads fail, but PCI configuration space is apparently still there:
> nvme nvme0: controller is down; will reset: CSTS=0xffffffff, PCI_STATUS=0x10
> and it comes with a nice AER dump:
> [12720.894411] pcieport 0000:00:1c.0: AER: Multiple Corrected error received: id=00e0
> [12720.909747] pcieport 0000:00:1c.0: PCIe Bus Error: severity=Corrected, type=Physical Layer, id=00e0(Transmitter ID)
> [12720.909751] pcieport 0000:00:1c.0: device [8086:9d14] error status/mask=00001001/00002000
> [12720.909754] pcieport 0000:00:1c.0: [ 0] Receiver Error (First)
> [12720.909756] pcieport 0000:00:1c.0: [12] Replay Timer Timeout
I opened this bugzilla and attached the dmesg and lspci -vv output to
it: https://bugzilla.kernel.org/show_bug.cgi?id=199695
The root port at 00:1c.0 leads to the NVMe device at 01:00.0 (this is
nvme0):
00:1c.0 PCI bridge: Intel Corporation Sunrise Point-LP PCI Express Root Port #5 (rev f1) (prog-if 00 [Normal decode])
Bus: primary=00, secondary=01, subordinate=01, sec-latency=0
01:00.0 Non-Volatile memory controller: Samsung Electronics Co Ltd NVMe SSD Controller SM961/PM961 (prog-if 02 [NVM Express])
Subsystem: Samsung Electronics Co Ltd Device a801
We reported several corrected errors before the nvme timeout:
[12750.281158] nvme nvme0: controller is down; will reset: CSTS=0xffffffff, PCI_STATUS=0x10
[12750.297594] nvme nvme0: I/O 455 QID 2 timeout, disable controller
[12750.305196] nvme 0000:01:00.0: enabling device (0000 -> 0002)
[12750.305465] nvme nvme0: Removing after probe failure status: -19
[12750.313188] nvme nvme0: I/O 456 QID 2 timeout, disable controller
[12750.329152] nvme nvme0: I/O 457 QID 2 timeout, disable controller
The corrected errors are supposedly recovered in hardware without
software intervention, and AER logs them for informational purposes.
But it seems very likely that these corrected errors are related to
the nvme timeout: the first corrected errors were logged at
12720.894411, nvme_io_timeout defaults to 30 seconds, and the nvme
timeout was at 12750.281158.
I don't have any good ideas. As a shot in the dark, you could try
running these commands before doing a suspend:
# setpci -s01:00.0 0x98.W
# setpci -s00:1c.0 0x68.W
# setpci -s01:00.0 0x198.L
# setpci -s00:1c.0 0x208.L
# setpci -s01:00.0 0x198.L=0x00000000
# setpci -s01:00.0 0x98.W=0x0000
# setpci -s00:1c.0 0x208.L=0x00000000
# setpci -s00:1c.0 0x68.W=0x0000
# lspci -vv -s00:1c.0
# lspci -vv -s01:00.0
The idea is to turn off ASPM L1.2 and LTR, just because that's new and
we've had issues with it before. If you try this, please collect the
output of the commands above in addition to the dmesg log, in case my
math is bad.
Bjorn
On Fri, May 11, 2018 at 11:57:52AM -0500, Bjorn Helgaas wrote:
> We reported several corrected errors before the nvme timeout:
>
> [12750.281158] nvme nvme0: controller is down; will reset: CSTS=0xffffffff, PCI_STATUS=0x10
> [12750.297594] nvme nvme0: I/O 455 QID 2 timeout, disable controller
> [12750.305196] nvme 0000:01:00.0: enabling device (0000 -> 0002)
> [12750.305465] nvme nvme0: Removing after probe failure status: -19
> [12750.313188] nvme nvme0: I/O 456 QID 2 timeout, disable controller
> [12750.329152] nvme nvme0: I/O 457 QID 2 timeout, disable controller
>
> The corrected errors are supposedly recovered in hardware without
> software intervention, and AER logs them for informational purposes.
>
> But it seems very likely that these corrected errors are related to
> the nvme timeout: the first corrected errors were logged at
> 12720.894411, nvme_io_timeout defaults to 30 seconds, and the nvme
> timeout was at 12750.281158.
The nvme_timeout handling is broken at the moment, but I'm not sure any
of the fixes being considered will help here if we're really getting
MMIO errors (that's what it looks like).
> I don't have any good ideas. As a shot in the dark, you could try
> running these commands before doing a suspend:
>
> # setpci -s01:00.0 0x98.W
> # setpci -s00:1c.0 0x68.W
> # setpci -s01:00.0 0x198.L
> # setpci -s00:1c.0 0x208.L
>
> # setpci -s01:00.0 0x198.L=0x00000000
> # setpci -s01:00.0 0x98.W=0x0000
> # setpci -s00:1c.0 0x208.L=0x00000000
> # setpci -s00:1c.0 0x68.W=0x0000
>
> # lspci -vv -s00:1c.0
> # lspci -vv -s01:00.0
>
> The idea is to turn off ASPM L1.2 and LTR, just because that's new and
> we've had issues with it before. If you try this, please collect the
> output of the commands above in addition to the dmesg log, in case my
> math is bad.
I trust you know the offsets here, but it's hard to tell what this
is doing with hard-coded addresses. Just to be safe and for clarity,
I recommend the 'CAP_*+<offset>' with a mask.
For example, disabling ASPM L1.2 can look like:
# setpci -s <B:D.f> CAP_PM+8.l=0:4
And disabling LTR:
# setpci -s <B:D.f> CAP_EXP+28.w=0:400
On Fri, May 11, 2018 at 11:26:11AM -0600, Keith Busch wrote:
> I trust you know the offsets here, but it's hard to tell what this
> is doing with hard-coded addresses. Just to be safe and for clarity,
> I recommend the 'CAP_*+<offset>' with a mask.
>
> For example, disabling ASPM L1.2 can look like:
>
> # setpci -s <B:D.f> CAP_PM+8.l=0:4
My mistake above: CAP_PM is a different capability, not the intended
one. It looks like setpci doesn't even have a convenient symbol for the
L1 PM extended capability, so the hard-coded offsets are the only way
for this setting. Sorry about the mistake.
On Fri, May 11, 2018 at 11:42:42AM -0600, Keith Busch wrote:
> On Fri, May 11, 2018 at 11:26:11AM -0600, Keith Busch wrote:
> > I trust you know the offsets here, but it's hard to tell what this
> > is doing with hard-coded addresses. Just to be safe and for clarity,
> > I recommend the 'CAP_*+<offset>' with a mask.
> >
> > For example, disabling ASPM L1.2 can look like:
> >
> > # setpci -s <B:D.f> CAP_PM+8.l=0:4
>
> My mistake above: CAP_PM is a different capability, not the intended
> one. It looks like setpci doesn't even have a convenient symbol for the
> L1 PM extended capability, so the hard-coded offsets are the only way
> for this setting. Sorry about the mistake.
No problem, would be a nice janitor job to add more of those symbolic
offsets to setpci. I recently added more cap ID decoding to lspci, but
didn't think about updating these things for setpci.
Using the symbols would definitely make this easier and better!
On Sat, May 12, 2018 at 12:57 AM, Bjorn Helgaas <[email protected]> wrote:
> Andrew wrote:
>> A friend of mine has a brand new LG laptop that has intermittent NVMe
>> failures. They mostly happen during a suspend/resume cycle
>> (apparently during suspend, not resume). Unlike the earlier
>> Dell/Samsung issue, the NVMe device isn't completely gone -- MMIO
>> reads fail, but PCI configuration space is apparently still there:
>
>> nvme nvme0: controller is down; will reset: CSTS=0xffffffff, PCI_STATUS=0x10
>
>> and it comes with a nice AER dump:
>
>> [12720.894411] pcieport 0000:00:1c.0: AER: Multiple Corrected error received: id=00e0
>> [12720.909747] pcieport 0000:00:1c.0: PCIe Bus Error: severity=Corrected, type=Physical Layer, id=00e0(Transmitter ID)
>> [12720.909751] pcieport 0000:00:1c.0: device [8086:9d14] error status/mask=00001001/00002000
>> [12720.909754] pcieport 0000:00:1c.0: [ 0] Receiver Error (First)
>> [12720.909756] pcieport 0000:00:1c.0: [12] Replay Timer Timeout
>
> I opened this bugzilla and attached the dmesg and lspci -vv output to
> it: https://bugzilla.kernel.org/show_bug.cgi?id=199695
>
> The root port at 00:1c.0 leads to the NVMe device at 01:00.0 (this is
> nvme0):
>
> 00:1c.0 PCI bridge: Intel Corporation Sunrise Point-LP PCI Express Root Port #5 (rev f1) (prog-if 00 [Normal decode])
> Bus: primary=00, secondary=01, subordinate=01, sec-latency=0
> 01:00.0 Non-Volatile memory controller: Samsung Electronics Co Ltd NVMe SSD Controller SM961/PM961 (prog-if 02 [NVM Express])
> Subsystem: Samsung Electronics Co Ltd Device a801
>
> We reported several corrected errors before the nvme timeout:
>
> [12750.281158] nvme nvme0: controller is down; will reset: CSTS=0xffffffff, PCI_STATUS=0x10
> [12750.297594] nvme nvme0: I/O 455 QID 2 timeout, disable controller
> [12750.305196] nvme 0000:01:00.0: enabling device (0000 -> 0002)
> [12750.305465] nvme nvme0: Removing after probe failure status: -19
> [12750.313188] nvme nvme0: I/O 456 QID 2 timeout, disable controller
> [12750.329152] nvme nvme0: I/O 457 QID 2 timeout, disable controller
>
> The corrected errors are supposedly recovered in hardware without
> software intervention, and AER logs them for informational purposes.
>
> But it seems very likely that these corrected errors are related to
> the nvme timeout: the first corrected errors were logged at
> 12720.894411, nvme_io_timeout defaults to 30 seconds, and the nvme
> timeout was at 12750.281158.
The following patchset might help this issue:
https://marc.info/?l=linux-block&m=152604179903505&w=2
--
Ming Lei