On Mon, Sep 20, 2010 at 04:15:32AM -0400, Gleb Natapov wrote:
> Nested SVM checks for external interrupt after injecting nested exception.
> In case there is external interrupt pending the code generates "external
> interrupt exit" and overwrites previous exit info. If previously injected
> exception already generated exit it will be lost.
> 
> Signed-off-by: Gleb Natapov <[email protected]>

Acked-by: Joerg Roedel <[email protected]>

> ---
>  v1->v2
>   - Added comment
> 
> diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
> index 43f5558..ca778d5 100644
> --- a/arch/x86/kvm/svm.c
> +++ b/arch/x86/kvm/svm.c
> @@ -1707,6 +1707,14 @@ static inline bool nested_svm_intr(struct vcpu_svm 
> *svm)
>       if (!(svm->vcpu.arch.hflags & HF_HIF_MASK))
>               return false;
>  
> +     /*
> +      * if vmexit was already requested (by intercepted exception
> +      * for instance) do not overwrite it with "external interrupt"
> +      * vmexit.
> +      */
> +     if (svm->nested.exit_required)
> +             return false;
> +
>       svm->vmcb->control.exit_code   = SVM_EXIT_INTR;
>       svm->vmcb->control.exit_info_1 = 0;
>       svm->vmcb->control.exit_info_2 = 0;
> --
>                       Gleb.
> 

-- 
AMD Operating System Research Center

Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to