Page MenuHomeFreeBSD
Differential D48583

arm64: add HCRX_EL2 register
ClosedPublic
Actions

Authored by harry.moulton_arm.com on Tue, Jan 21, 5:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 1, 7:01 PM
Unknown Object (File)
Sat, Feb 1, 5:18 AM
Unknown Object (File)
Fri, Jan 31, 8:10 AM
Unknown Object (File)
Fri, Jan 31, 5:55 AM
Unknown Object (File)
Fri, Jan 24, 12:26 AM
Subscribers
emaste
imp

Details

Reviewers
andrew
manu
Group Reviewers
arm64
Commits
rG4d1acfb132a5: arm64: add HCRX_EL2 register
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 Not Applicable
Unit
Tests Not Applicable

Event Timeline

harry.moulton_arm.com created this revision.Tue, Jan 21, 5:47 PM
Herald added a reviewer: manu. · View Herald TranscriptTue, Jan 21, 5:47 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
harry.moulton_arm.com requested review of this revision.Tue, Jan 21, 5:47 PM
Harbormaster completed remote builds in B61872: Diff 149676.Tue, Jan 21, 5:47 PM
harry.moulton_arm.com added a child revision: D48584: arm64/vmm: save and restore HCRX_EL2 register.Tue, Jan 21, 5:47 PM
andrew added inline comments.Tue, Jan 21, 6:17 PM
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.

harry.moulton_arm.com updated this revision to Diff 149706.Wed, Jan 22, 2:02 PM

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

Harbormaster completed remote builds in B61880: Diff 149706.Wed, Jan 22, 2:02 PM
andrew added inline comments.Wed, Jan 22, 2:38 PM
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/

harry.moulton_arm.com updated this revision to Diff 149711.Wed, Jan 22, 3:21 PM

Move configuration of hcrx_el2 to avoid trashing cptr_el2

Harbormaster completed remote builds in B61882: Diff 149711.Wed, Jan 22, 3:21 PM
andrew accepted this revision.Wed, Jan 22, 3:41 PM
This revision is now accepted and ready to land.Wed, Jan 22, 3:41 PM
Closed by commit rG4d1acfb132a5: arm64: add HCRX_EL2 register (authored by harry.moulton_arm.com, committed by andrew). · Explain WhyThu, Jan 23, 5:28 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG4d1acfb132a5: arm64: add HCRX_EL2 register.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
19 lines
include/
35 lines

Diff 149806

View Options

sys/arm64/arm64/locore.S

Loading...
View Options

sys/arm64/include/hypervisor.h

Loading...