KVM: SVM: Zero out GDTR.base and IDTR.base on INIT
authorSean Christopherson <seanjc@google.com>
Tue, 13 Jul 2021 16:32:41 +0000 (09:32 -0700)
committerPaolo Bonzini <pbonzini@redhat.com>
Mon, 2 Aug 2021 15:01:49 +0000 (11:01 -0400)
Explicitly set GDTR.base and IDTR.base to zero when intializing the VMCB.
Functionally this only affects INIT, as the bases are implicitly set to
zero on RESET by virtue of the VMCB being zero allocated.

Per AMD's APM, GDTR.base and IDTR.base are zeroed after RESET and INIT.

Fixes: 04d2cc7780d4 ("KVM: Move main vcpu loop into subarch independent code")
Signed-off-by: Sean Christopherson <seanjc@google.com>
Message-Id: <20210713163324.627647-4-seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/svm/svm.c

index 4ce6d827fccd48dd597f5c3d618d5b219da19c64..7845232b6fb686c19ca0c2a45ebaee786595136e 100644 (file)
@@ -1241,7 +1241,9 @@ static void init_vmcb(struct kvm_vcpu *vcpu)
                SVM_SELECTOR_S_MASK | SVM_SELECTOR_CODE_MASK;
        save->cs.limit = 0xffff;
 
+       save->gdtr.base = 0;
        save->gdtr.limit = 0xffff;
+       save->idtr.base = 0;
        save->idtr.limit = 0xffff;
 
        init_sys_seg(&save->ldtr, SEG_TYPE_LDT);