Commit 6f57c6be authored by Marc Zyngier's avatar Marc Zyngier
Browse files

KVM: arm64: nv: Drop VCPU_HYP_CONTEXT flag 



It has become obvious that HCR_EL2.NV serves the exact same use
as VCPU_HYP_CONTEXT, only in an architectural way. So just drop
the flag for good.

Reviewed-by: default avatarJoey Gouly <joey.gouly@arm.com>
Reviewed-by: default avatarOliver Upton <oliver.upton@linux.dev>
Link: https://lore.kernel.org/r/20240419102935.1935571-5-maz@kernel.org


Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
parent a07e9345

arch/arm64/include/asm/kvm_host.h

+0 −2
Original line number Diff line number Diff line
@@ -817,8 +817,6 @@ struct kvm_vcpu_arch { 
#define DEBUG_STATE_SAVE_SPE	__vcpu_single_flag(iflags, BIT(5))

/* Save TRBE context if active  */

#define DEBUG_STATE_SAVE_TRBE	__vcpu_single_flag(iflags, BIT(6))

/* vcpu running in HYP context */

#define VCPU_HYP_CONTEXT	__vcpu_single_flag(iflags, BIT(7))



/* SVE enabled for host EL0 */

#define HOST_SVE_ENABLED	__vcpu_single_flag(sflags, BIT(0))

arch/arm64/kvm/hyp/vhe/switch.c

+1 −6
Original line number Diff line number Diff line
@@ -197,7 +197,7 @@ static void early_exit_filter(struct kvm_vcpu *vcpu, u64 *exit_code) 
	 * If we were in HYP context on entry, adjust the PSTATE view

	 * so that the usual helpers work correctly.

	 */

	if (unlikely(vcpu_get_flag(vcpu, VCPU_HYP_CONTEXT))) {

	if (vcpu_has_nv(vcpu) && (read_sysreg(hcr_el2) & HCR_NV)) {

		u64 mode = *vcpu_cpsr(vcpu) & (PSR_MODE_MASK | PSR_MODE32_BIT);



		switch (mode) {
@@ -240,11 +240,6 @@ static int __kvm_vcpu_run_vhe(struct kvm_vcpu *vcpu) 
	sysreg_restore_guest_state_vhe(guest_ctxt);

	__debug_switch_to_guest(vcpu);



	if (is_hyp_ctxt(vcpu))

		vcpu_set_flag(vcpu, VCPU_HYP_CONTEXT);

	else

		vcpu_clear_flag(vcpu, VCPU_HYP_CONTEXT);



	do {

		/* Jump in the fire! */

		exit_code = __guest_enter(vcpu);