Re: [PATCH v2 0/3] Fix task switches into/out of VM86

[Gleb Natapov <gleb@redhat.com>]

On Fri, Jan 27, 2012 at 08:23:33PM +0100, Kevin Wolf wrote:
> I believe this should work with both VMX and SVM now. Gleb, Jörg, can one of
> you test this with SVM? I did some testing on my buggy processor and it looks
> as good as it gets, but it would be better if you could confirm.
> 
You forgot to set cpl to 3 in vmcb in svm_set_rflags() when vm86 is enabled, no?

> Kevin Wolf (3):
>   KVM: x86 emulator: Fix task switch privilege checks
>   KVM: x86 emulator: VM86 segments must have DPL 3
>   KVM: x86 emulator: Allow PM/VM86 switch during task switch
> 
>  arch/x86/include/asm/kvm_emulate.h |    3 +-
>  arch/x86/include/asm/kvm_host.h    |    4 +-
>  arch/x86/kvm/emulate.c             |   79 ++++++++++++++++++++++++++++++++---
>  arch/x86/kvm/svm.c                 |    5 ++-
>  arch/x86/kvm/vmx.c                 |    8 ++-
>  arch/x86/kvm/x86.c                 |   12 ++++-
>  6 files changed, 94 insertions(+), 17 deletions(-)
> 
> -- 
> 1.7.6.5

--
			Gleb.