mirror of
https://github.com/torvalds/linux.git
synced 2024-11-29 15:41:36 +00:00
KVM: VMX: Modify NMI and INTR handlers to take intr_info as function argument
TDX uses different ABI to get information about VM exit. Pass intr_info to the NMI and INTR handlers instead of pulling it from vcpu_vmx in preparation for sharing the bulk of the handlers with TDX. When the guest TD exits to VMM, RAX holds status and exit reason, RCX holds exit qualification etc rather than the VMCS fields because VMM doesn't have access to the VMCS. The eventual code will be VMX: - get exit reason, intr_info, exit_qualification, and etc from VMCS - call NMI/INTR handlers (common code) TDX: - get exit reason, intr_info, exit_qualification, and etc from guest registers - call NMI/INTR handlers (common code) Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Message-Id: <0396a9ae70d293c9d0b060349dae385a8a4fbcec.1705965635.git.isaku.yamahata@intel.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
5f18c642ff
commit
2325a21ac1
@ -6956,24 +6956,22 @@ static void handle_nm_fault_irqoff(struct kvm_vcpu *vcpu)
|
||||
rdmsrl(MSR_IA32_XFD_ERR, vcpu->arch.guest_fpu.xfd_err);
|
||||
}
|
||||
|
||||
static void handle_exception_irqoff(struct vcpu_vmx *vmx)
|
||||
static void handle_exception_irqoff(struct kvm_vcpu *vcpu, u32 intr_info)
|
||||
{
|
||||
u32 intr_info = vmx_get_intr_info(&vmx->vcpu);
|
||||
|
||||
/* if exit due to PF check for async PF */
|
||||
if (is_page_fault(intr_info))
|
||||
vmx->vcpu.arch.apf.host_apf_flags = kvm_read_and_reset_apf_flags();
|
||||
vcpu->arch.apf.host_apf_flags = kvm_read_and_reset_apf_flags();
|
||||
/* if exit due to NM, handle before interrupts are enabled */
|
||||
else if (is_nm_fault(intr_info))
|
||||
handle_nm_fault_irqoff(&vmx->vcpu);
|
||||
handle_nm_fault_irqoff(vcpu);
|
||||
/* Handle machine checks before interrupts are enabled */
|
||||
else if (is_machine_check(intr_info))
|
||||
kvm_machine_check();
|
||||
}
|
||||
|
||||
static void handle_external_interrupt_irqoff(struct kvm_vcpu *vcpu)
|
||||
static void handle_external_interrupt_irqoff(struct kvm_vcpu *vcpu,
|
||||
u32 intr_info)
|
||||
{
|
||||
u32 intr_info = vmx_get_intr_info(vcpu);
|
||||
unsigned int vector = intr_info & INTR_INFO_VECTOR_MASK;
|
||||
|
||||
if (KVM_BUG(!is_external_intr(intr_info), vcpu->kvm,
|
||||
@ -6998,9 +6996,9 @@ void vmx_handle_exit_irqoff(struct kvm_vcpu *vcpu)
|
||||
return;
|
||||
|
||||
if (vmx->exit_reason.basic == EXIT_REASON_EXTERNAL_INTERRUPT)
|
||||
handle_external_interrupt_irqoff(vcpu);
|
||||
handle_external_interrupt_irqoff(vcpu, vmx_get_intr_info(vcpu));
|
||||
else if (vmx->exit_reason.basic == EXIT_REASON_EXCEPTION_NMI)
|
||||
handle_exception_irqoff(vmx);
|
||||
handle_exception_irqoff(vcpu, vmx_get_intr_info(vcpu));
|
||||
}
|
||||
|
||||
/*
|
||||
|
Loading…
Reference in New Issue
Block a user