]> www.infradead.org Git - users/jedix/linux-maple.git/commit
KVM: arm64: Force HCR_EL2.xMO to 1 at all times in VHE mode
authorMarc Zyngier <maz@kernel.org>
Tue, 29 Apr 2025 11:43:26 +0000 (12:43 +0100)
committerMarc Zyngier <maz@kernel.org>
Tue, 6 May 2025 08:41:32 +0000 (09:41 +0100)
commitbae247ccade0a5016031e73c9f6d61b758b612d8
tree68136e22510f3fd67911dfc85384b66a7a0ecb53
parentd2f14174f9e839e1128664ab339bbe17c333208b
KVM: arm64: Force HCR_EL2.xMO to 1 at all times in VHE mode

We keep setting and clearing these bits depending on the role of
the host kernel, mimicking what we do for nVHE. But that's actually
pretty pointless, as we always want physical interrupts to make it
to the host, at EL2.

This has also two problems:

- it prevents IRQs from being taken when these bits are cleared
  if the implementation has chosen to implement these bits as
  masks when HCR_EL2.{TGE,xMO}=={0,0}

- it triggers a bad erratum on the AmpereOne HW, which catches
  fire on clearing these bits while an interrupt is being taken
  (AC03_CPU_36).

Let's kill these two birds with a single stone, and permanently
set the xMO bits when running VHE. This involves a bit of surgery
on code paths that rely on flipping these bits on and off for
other purposes.

Note that the earliest setting of hcr_el2 (in the init_hcr_el2
macro) is left untouched as is runs extremely early, with interrupts
disabled, and soon enough overwritten with the final value containing
the xMO bits.

Reported-by: D Scott Phillips <scott@os.amperecomputing.com>
Link: https://lore.kernel.org/r/20250429114326.3618875-1-maz@kernel.org
Signed-off-by: Marc Zyngier <maz@kernel.org>
arch/arm64/include/asm/kvm_arm.h
arch/arm64/kvm/hyp/vgic-v3-sr.c