On Tue, Feb 08, 2011 at 07:22:29PM -0500, Andre Przywara wrote:
> Somehow the code line advancing the RIP and checking for exceptions
> got dropped between the post on the ML and the commit.
> Add it again to let guests boot on upcoming AMD CPUs again.
>
> Reported-by: Joerg Roedel <[email protected]>
> Signed-off-by: Andre Przywara <[email protected]>
> ---
> arch/x86/kvm/svm.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
> index 25bd1bc..be06e58 100644
> --- a/arch/x86/kvm/svm.c
> +++ b/arch/x86/kvm/svm.c
> @@ -2776,6 +2776,7 @@ static int dr_interception(struct vcpu_svm *svm)
> if (!err)
> kvm_register_write(&svm->vcpu, reg, val);
> }
> + kvm_complete_insn_gp(&svm->vcpu, err);
Any exception check takes presedence over the intercept. So a call to
skip_emulated_instruction should be sufficient here.
Joerg
--
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
