AMD

Replace AMD nvme workaround from oem-5.10 with upstream version

Bug #1930719 reported by Mario Limonciello
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
AMD
Fix Released
Undecided
Unassigned
linux-oem-5.10 (Ubuntu)
Invalid
Undecided
Unassigned
Focal
Won't Fix
Undecided
Unassigned
Impish
Invalid
Undecided
Unassigned
linux-oem-5.13 (Ubuntu)
Invalid
Undecided
Unassigned
Focal
Fix Released
Undecided
Unassigned
Impish
Invalid
Undecided
Unassigned

Bug Description

The workaround carried in linux-oem-5.10 to support suspend to idle on AMD platforms:

https://kernel.ubuntu.com/git/kernel-ppa/mirror/ubuntu-oem-5.10-focal.git/commit/drivers/nvme?h=oem-5.10-next&id=2d37b035a0f469d40ab0322b0f9dd7f2d43bc9b8

Should be replaced with what's going upstream for 5.14:
http://git.infradead.org/nvme.git/commit/e21e0243e7b0f1c2a21d21f4d115f7b37175772a

And please also apply it to oem-5.13.

CVE References

information type: Proprietary → Private
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

Can I turn it public?

Changed in linux-oem-5.10 (Ubuntu):
assignee: nobody → Kai-Heng Feng (kaihengfeng)
Revision history for this message
Mario Limonciello (superm1) wrote :

Yes

Changed in amd:
status: New → Fix Committed
information type: Private → Public
Revision history for this message
Mario Limonciello (superm1) wrote :

It seems that some OEM designs don't have StorageD3enable set in the acpi tables for Renoir.

So we should use this for Cezanne, but may need another quirk elsewhere for RN. So please don't remove the sauce patches until we come up with another idea for them.

Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

OK, I'll wait until it settles down.

Revision history for this message
Mario Limonciello (superm1) wrote :

Given NVME upstream's reluctance to accept any sort of #ifdef CONFIG_X86 or match against a PCI Renoir device for a quirk, I'm not sure where the right home for that behavior can be done for Renoir without BIOS change.

Timo Aaltonen (tjaalton)
Changed in linux-oem-5.10 (Ubuntu):
status: New → Invalid
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

I am not sure why it's needed? This can be totally done in nvme driver...

Revision history for this message
Mario Limonciello (superm1) wrote :

It can be, but Christoph wants this out of the NVME driver. It can technically apply to any storage device. So there are other patches that will move it out and add quirk there:
https://patchwork.kernel.org/project/linux-acpi/list/?series=496353

Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

IMO it's a bit over-engineered, but it's up to maintainers to decide...

Revision history for this message
Mario Limonciello (superm1) wrote :
Revision history for this message
Mario Limonciello (superm1) wrote :

Can you please pick up these 3 for oem-5.10 and oem-5.13?

Changed in linux-oem-5.13 (Ubuntu):
status: New → Invalid
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

Sending the fix in another SRU.

Changed in linux-oem-5.10 (Ubuntu Focal):
status: New → Won't Fix
Changed in linux-oem-5.13 (Ubuntu Focal):
status: New → Won't Fix
Revision history for this message
Mario Limonciello (superm1) wrote :

Looks like other SRU missed #9 still.

Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

Right, I was being sloppy.

Changed in linux-oem-5.10 (Ubuntu Focal):
status: Won't Fix → Confirmed
Changed in linux-oem-5.13 (Ubuntu Focal):
status: Won't Fix → Confirmed
Changed in linux-oem-5.10 (Ubuntu Focal):
assignee: nobody → Kai-Heng Feng (kaihengfeng)
Changed in linux-oem-5.13 (Ubuntu Focal):
assignee: nobody → Kai-Heng Feng (kaihengfeng)
Changed in linux-oem-5.10 (Ubuntu):
assignee: Kai-Heng Feng (kaihengfeng) → nobody
information type: Public → Public Security
information type: Public Security → Public
Timo Aaltonen (tjaalton)
Changed in linux-oem-5.13 (Ubuntu Focal):
status: Confirmed → Fix Committed
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-focal' to 'verification-done-focal'. If the problem still exists, change the tag 'verification-needed-focal' to 'verification-failed-focal'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-focal
tags: added: verification-done-focal
removed: verification-needed-focal
tags: added: patch
Changed in linux-oem-5.10 (Ubuntu Focal):
assignee: Kai-Heng Feng (kaihengfeng) → nobody
Changed in linux-oem-5.13 (Ubuntu Focal):
assignee: Kai-Heng Feng (kaihengfeng) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.4 KiB)

This bug was fixed in the package linux-oem-5.13 - 5.13.0-1009.10

---------------
linux-oem-5.13 (5.13.0-1009.10) focal; urgency=medium

  * CVE-2021-33909
    - SAUCE: seq_file: Disallow extremely large seq buffer allocations

linux-oem-5.13 (5.13.0-1007.7) focal; urgency=medium

  * focal/linux-oem-5.13: 5.13.0-1007.7 -proposed tracker (LP: #1934979)

  * intel_pmc_core driver is not probed on ADL (LP: #1934660)
    - SAUCE: platform/x86: intel_pmc_core: Add Alderlake support to pmc_core
      driver
    - SAUCE: platform/x86: intel_pmc_core: Add Latency Tolerance Reporting (LTR)
      support to Alder Lake
    - SAUCE: platform/x86: intel_pmc_core: Add Alder Lake low power mode support
      for pmc_core
    - SAUCE: platform/x86: intel_pmc_core: Add GBE Package C10 fix for Alder Lake
      PCH

linux-oem-5.13 (5.13.0-1006.6) focal; urgency=medium

  * focal/linux-oem-5.13: 5.13.0-1006.6 -proposed tracker (LP: #1934490)

  * Replace AMD nvme workaround from oem-5.10 with upstream version
    (LP: #1930719)
    - ACPI: Check StorageD3Enable _DSD property in ACPI code
    - ACPI: Add quirks for AMD Renoir/Lucienne CPUs to force the D3 hint

  * Add Thunderbolt support for Intel Alder Lake (LP: #1934240)
    - thunderbolt: Add support for Intel Alder Lake

  * Fix Ethernet not working by hotplug - RTL8106E (LP: #1930645)
    - SAUCE: r8169: Use PHY_POLL when RTL8106E enable ASPM

  * Realtek USB hubs in Dell WD19SC/DC/TB fail to work after exiting s2idle
    (LP: #1928242)
    - USB: Verify the port status when timeout happens during port suspend
    - Revert "USB: Add reset-resume quirk for WD19's Realtek Hub"

  * USB Type-C hotplug event not handled properly in TGL-H system during s2idle
    (LP: #1931072)
    - drm/i915: Force a TypeC PHY disconnect during suspend/shutdown

  * Miscellaneous Ubuntu changes
    - [Config] Sync annotations and config with master

  [ Ubuntu: 5.13.0-11.11 ]

  * impish/linux: 5.13.0-11.11 -proposed tracker (LP: #1933854)
  * Packaging resync (LP: #1786013)
    - [Packaging] update variants
  * Support builtin revoked certificates (LP: #1932029)
    - [Packaging] build canonical-revoked-certs.pem from branch/arch certs
    - [Packaging] Revoke 2012 UEFI signing certificate as built-in
    - [Config] Configure CONFIG_SYSTEM_REVOCATION_KEYS with revoked keys
  * Miscellaneous Ubuntu changes
    - [Packaging] Change source package name to linux
  * Miscellaneous upstream changes
    - mm/page_alloc: Correct return value of populated elements if bulk array is
      populated

  [ Ubuntu: 5.13.0-10.10 ]

  * impish/linux-unstable: 5.13.0-10.10 -proposed tracker (LP: #1933795)
  * Pixel format change broken for Elgato Cam Link 4K (LP: #1932367)
    - media: uvcvideo: Fix pixel format change for Elgato Cam Link 4K
  * initramfs-tools & kernel: use zstd as the default compression method
    (LP: #1931725)
    - s390/decompressor: correct BOOT_HEAP_SIZE condition
    - s390/boot: add zstd support
    - [Packaging] use ZSTD to compress s390 kernels
  * Miscellaneous Ubuntu changes
    - SAUCE: selftests: tls: fix chacha+bidir tests
    - SAUCE: selftests: icmp_redirect: support expected failures
    - [Config] update ...

Read more...

Changed in linux-oem-5.13 (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. While running an Ubuntu kernel (not a mainline or third-party kernel) please enter the following command in a terminal window:

apport-collect 1930719

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Changed in linux (Ubuntu Focal):
status: New → Incomplete
no longer affects: linux (Ubuntu)
no longer affects: linux (Ubuntu Focal)
no longer affects: linux (Ubuntu Impish)
Changed in linux-oem-5.10 (Ubuntu Focal):
status: Confirmed → Won't Fix
Changed in amd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.