Skip to content
Permalink
Browse files
KVM: arm64: Guest exit handlers for nVHE hyp
Add an array of pointers to handlers for various trap reasons in
nVHE code.

The current code selects how to fixup a guest on exit based on a
series of if/else statements. Future patches will also require
different handling for guest exists. Create an array of handlers
to consolidate them.

No functional change intended as the array isn't populated yet.

Acked-by: Will Deacon <will@kernel.org>
Signed-off-by: Fuad Tabba <tabba@google.com>
  • Loading branch information
Fuad Tabba authored and intel-lab-lkp committed Aug 17, 2021
1 parent c3575d2 commit d20adc963041fb77c56e8da7935236d73b738e63
Show file tree
Hide file tree
Showing 2 changed files with 76 additions and 0 deletions.
@@ -409,6 +409,46 @@ static inline bool __hyp_handle_ptrauth(struct kvm_vcpu *vcpu)
return true;
}

typedef int (*exit_handle_fn)(struct kvm_vcpu *);

exit_handle_fn kvm_get_nvhe_exit_handler(struct kvm_vcpu *vcpu);

static exit_handle_fn kvm_get_hyp_exit_handler(struct kvm_vcpu *vcpu)
{
return is_nvhe_hyp_code() ? kvm_get_nvhe_exit_handler(vcpu) : NULL;
}

/*
* Allow the hypervisor to handle the exit with an exit handler if it has one.
*
* Returns true if the hypervisor handled the exit, and control should go back
* to the guest, or false if it hasn't.
*/
static bool kvm_hyp_handle_exit(struct kvm_vcpu *vcpu)
{
bool is_handled = false;
exit_handle_fn exit_handler = kvm_get_hyp_exit_handler(vcpu);

if (exit_handler) {
/*
* There's limited vcpu context here since it's not synced yet.
* Ensure that relevant vcpu context that might be used by the
* exit_handler is in sync before it's called and if handled.
*/
*vcpu_pc(vcpu) = read_sysreg_el2(SYS_ELR);
*vcpu_cpsr(vcpu) = read_sysreg_el2(SYS_SPSR);

is_handled = exit_handler(vcpu);

if (is_handled) {
write_sysreg_el2(*vcpu_pc(vcpu), SYS_ELR);
write_sysreg_el2(*vcpu_cpsr(vcpu), SYS_SPSR);
}
}

return is_handled;
}

/*
* Return true when we were able to fixup the guest exit and should return to
* the guest, false when we should restore the host state and return to the
@@ -496,6 +536,9 @@ static inline bool fixup_guest_exit(struct kvm_vcpu *vcpu, u64 *exit_code)
goto guest;
}

/* Check if there's an exit handler and allow it to handle the exit. */
if (kvm_hyp_handle_exit(vcpu))
goto guest;
exit:
/* Return to the host kernel and handle the exit */
return false;
@@ -158,6 +158,39 @@ static void __pmu_switch_to_host(struct kvm_cpu_context *host_ctxt)
write_sysreg(pmu->events_host, pmcntenset_el0);
}

static exit_handle_fn hyp_exit_handlers[] = {
[0 ... ESR_ELx_EC_MAX] = NULL,
[ESR_ELx_EC_WFx] = NULL,
[ESR_ELx_EC_CP15_32] = NULL,
[ESR_ELx_EC_CP15_64] = NULL,
[ESR_ELx_EC_CP14_MR] = NULL,
[ESR_ELx_EC_CP14_LS] = NULL,
[ESR_ELx_EC_CP14_64] = NULL,
[ESR_ELx_EC_HVC32] = NULL,
[ESR_ELx_EC_SMC32] = NULL,
[ESR_ELx_EC_HVC64] = NULL,
[ESR_ELx_EC_SMC64] = NULL,
[ESR_ELx_EC_SYS64] = NULL,
[ESR_ELx_EC_SVE] = NULL,
[ESR_ELx_EC_IABT_LOW] = NULL,
[ESR_ELx_EC_DABT_LOW] = NULL,
[ESR_ELx_EC_SOFTSTP_LOW] = NULL,
[ESR_ELx_EC_WATCHPT_LOW] = NULL,
[ESR_ELx_EC_BREAKPT_LOW] = NULL,
[ESR_ELx_EC_BKPT32] = NULL,
[ESR_ELx_EC_BRK64] = NULL,
[ESR_ELx_EC_FP_ASIMD] = NULL,
[ESR_ELx_EC_PAC] = NULL,
};

exit_handle_fn kvm_get_nvhe_exit_handler(struct kvm_vcpu *vcpu)
{
u32 esr = kvm_vcpu_get_esr(vcpu);
u8 esr_ec = ESR_ELx_EC(esr);

return hyp_exit_handlers[esr_ec];
}

/* Switch to the guest for legacy non-VHE systems */
int __kvm_vcpu_run(struct kvm_vcpu *vcpu)
{

0 comments on commit d20adc9

Please sign in to comment.