Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752265AbcDVTQk (ORCPT ); Fri, 22 Apr 2016 15:16:40 -0400 Received: from inet-orm.provo.novell.com ([137.65.248.124]:39554 "EHLO mail.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751850AbcDVTQj (ORCPT ); Fri, 22 Apr 2016 15:16:39 -0400 X-Greylist: delayed 1216 seconds by postgrey-1.27 at vger.kernel.org; Fri, 22 Apr 2016 15:16:39 EDT From: Bruce Rogers To: kvm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: namit@cs.technion.ac.il, stable@vger.kernel.org, Bruce Rogers Subject: [PATCH] KVM: x86: fix ordering of cr0 initialization code in vmx_cpu_reset Date: Fri, 22 Apr 2016 12:56:03 -0600 Message-Id: <1461351363-28094-1-git-send-email-brogers@suse.com> X-Mailer: git-send-email 1.9.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1181 Lines: 31 Commit d28bc9dd25ce reversed the order of two lines which initialize cr0, allowing the current (old) cr0 value to mess up vcpu initialization. This was observed in the checks for cr0 X86_CR0_WP bit in the context of kvm_mmu_reset_context(). Besides, setting vcpu->arch.cr0 after vmx_set_cr0() is completely redundant. Change the order back to ensure proper vcpu initialization. The combination of booting with ovmf firmware when guest vcpus > 1 and kvm's ept=N option being set results in a VM-entry failure. This patch fixes that. Signed-off-by: Bruce Rogers --- arch/x86/kvm/vmx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index ee1c8a9..ab4a387 100644 --- a/arch/x86/kvm/vmx.c +++ b/arch/x86/kvm/vmx.c @@ -5046,8 +5046,8 @@ static void vmx_vcpu_reset(struct kvm_vcpu *vcpu, bool init_event) vmcs_write16(VIRTUAL_PROCESSOR_ID, vmx->vpid); cr0 = X86_CR0_NW | X86_CR0_CD | X86_CR0_ET; - vmx_set_cr0(vcpu, cr0); /* enter rmode */ vmx->vcpu.arch.cr0 = cr0; + vmx_set_cr0(vcpu, cr0); /* enter rmode */ vmx_set_cr4(vcpu, 0); vmx_set_efer(vcpu, 0); vmx_fpu_activate(vcpu); -- 1.9.0