Page MenuHomeFreeBSD

arm64: add HCRX_EL2 register
ClosedPublic

Authored by harry.moulton_arm.com on Jan 21 2025, 5:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 9, 7:28 AM
Unknown Object (File)
Mon, Dec 1, 5:01 PM
Unknown Object (File)
Thu, Nov 27, 5:33 PM
Unknown Object (File)
Fri, Nov 21, 10:27 PM
Unknown Object (File)
Wed, Nov 19, 4:31 AM
Unknown Object (File)
Tue, Nov 18, 6:21 AM
Unknown Object (File)
Mon, Nov 17, 3:33 PM
Unknown Object (File)
Thu, Nov 13, 1:31 AM
Subscribers

Details

Summary

HCRX_EL2 is the Extended Hypervisor Configuration Register introduced
with FEAT_HCX in ARMv8.7. All fields in this register are used for
features in ARMv8.7 and above. Initially zero the register, incase
firmware has not properly configured it.

Sponsored By: Arm Ltd
Signed-off-by: Harry Moulton <harry.moulton@arm.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 61882
Build 58766: arc lint + arc unit

Event Timeline

sys/arm64/arm64/locore.S
378–390

Is there a reason to put this in a VHE only code path? It's also unlikely to need the isb as there is one before the eret that can be used as the context synchronization event.

sys/arm64/include/armreg.h
721–727 ↗(On Diff #149676)

This should be in hypervisor.h as it's an EL2 only register.

Moved HCRX register definition to hypervisor.h, moved initial
configuration of hcrx_el2 to after .Ldone_vhe

sys/arm64/arm64/locore.S
400

This will trash the cptr_el2 value. After msr spsr_el2, x5 will be safe as that's the last register set based on VHE vs non-VHE/

Move configuration of hcrx_el2 to avoid trashing cptr_el2

This revision is now accepted and ready to land.Jan 22 2025, 3:41 PM
This revision was automatically updated to reflect the committed changes.