[Bug 2072381] Update Released
Andreas Hasenack
2072381 at bugs.launchpad.net
Thu Sep 12 20:51:39 UTC 2024
The verification of the Stable Release Update for nvme-cli has completed
successfully and the package is now being released to -updates.
Subsequently, the Ubuntu Stable Release Updates Team is being
unsubscribed and will not receive messages about this bug report. In
the event that you encounter a regression using the package from
-updates please report a new bug using ubuntu-bug and tag the bug report
regression-update so we can easily find any regressions.
--
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to nvme-cli in Ubuntu.
https://bugs.launchpad.net/bugs/2072381
Title:
NVMe show-regs command forcibly reboots ARM-based VM
Status in nvme-cli package in Ubuntu:
Fix Released
Status in nvme-cli source package in Focal:
Fix Released
Status in nvme-cli source package in Jammy:
Fix Released
Status in nvme-cli source package in Mantic:
Won't Fix
Status in nvme-cli source package in Noble:
Fix Released
Status in nvme-cli source package in Oracular:
Fix Released
Bug description:
[Impact]
The ARM-based VM will reboot after executing the 'nvme show-regs' command.
[Fix]
There is an upstream commit that fixed this issue:
commit fa5a46cc25b884a61a14f06e6693373bcefff29d
Author: Oliver Upton <oliver.upton at linux.dev>
Date: Thu Jul 4 19:59:26 2024 +0200
common.h: Avoid using unsupported load/store instructions in arm64
VMs
Using nvme show-regs within a VM on arm64 can sometimes lead to VM
termination.
To answer why this happens: one of the deficiencies of the Arm
architecture is that there exists a range of load/store instructions
that have insufficient decode information for traps taken to the
hypervisor. KVM, for example, may raise an external abort or outright
terminate the VM depending on the configuration.
This is a known problem on the kernel side, and is fixed by using
assembly MMIO accessors w/ 'safe' load/store instructions. So do
exactly that, providing arm64-specific accessors and falling back to
plain old volatile pointer accesses for other architectures.
Reported-by: William Butler <wab at google.com>
Signed-off-by: Oliver Upton <oliver.upton at linux.dev>
[Test Plan]
Execute the command 'nvme show-regs /dev/nvme0' to confirm if it functions correctly.
[Where problems could occur]
The patch fixes the issue by using assembly code to access MMIO on ARM, while keeping the original approach on other platforms. If the patch encounters any issues, it will only impact the functionality of displaying register data on ARM-based platforms.
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/nvme-cli/+bug/2072381/+subscriptions
More information about the foundations-bugs
mailing list