diff mbox series

[04/45] KVM: VMX: Allocate VPID after initializing VCPU

Message ID	20191022015925.31916-5-sean.j.christopherson@intel.com (mailing list archive)
State	New, archived
Headers	show Return-Path: <SRS0=/a1V=YP=lists.infradead.org=linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@kernel.org> DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 74A0E2064B From: Sean Christopherson <sean.j.christopherson@intel.com> To: Marc Zyngier <maz@kernel.org>, James Hogan <jhogan@kernel.org>, Paul Mackerras <paulus@ozlabs.org>, Christian Borntraeger <borntraeger@de.ibm.com>, Janosch Frank <frankja@linux.ibm.com>, Paolo Bonzini <pbonzini@redhat.com>, =?utf-8?b?UmFkaW0gS3LEjW3DocWZ?= <rkrcmar@redhat.com> Subject: [PATCH 04/45] KVM: VMX: Allocate VPID after initializing VCPU Date: Mon, 21 Oct 2019 18:58:44 -0700 Message-Id: <20191022015925.31916-5-sean.j.christopherson@intel.com> In-Reply-To: <20191022015925.31916-1-sean.j.christopherson@intel.com> References: <20191022015925.31916-1-sean.j.christopherson@intel.com> MIME-Version: 1.0 summary: Content analysis details: (-2.3 points) pts rule name description ---- ---------------------- -------------------------------------------------- -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at https://www.dnswl.org/, medium trust [134.134.136.100 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record Precedence: list Cc: kvm-ppc@vger.kernel.org, Wanpeng Li <wanpengli@tencent.com>, kvm@vger.kernel.org, David Hildenbrand <david@redhat.com>, Joerg Roedel <joro@8bytes.org>, Cornelia Huck <cohuck@redhat.com>, linux-mips@vger.kernel.org, Sean Christopherson <sean.j.christopherson@intel.com>, linux-kernel@vger.kernel.org, James Morse <james.morse@arm.com>, linux-arm-kernel@lists.infradead.org, Vitaly Kuznetsov <vkuznets@redhat.com>, Suzuki K Poulose <suzuki.poulose@arm.com>, kvmarm@lists.cs.columbia.edu, Julien Thierry <julien.thierry.kdev@gmail.com>, Jim Mattson <jmattson@google.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org> Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org
Series	KVM: Refactor vCPU creation \| expand [00/45] KVM: Refactor vCPU creation [01/45] KVM: PPC: Book3S HV: Uninit vCPU if vcore creation fails [02/45] KVM: PPC: Book3S PR: Free shared page if mmu initialization fails [03/45] KVM: x86: Free wbinvd_dirty_mask if vCPU creation fails [04/45] KVM: VMX: Allocate VPID after initializing VCPU [05/45] KVM: VMX: Use direct vcpu pointer during vCPU create/free [06/45] KVM: SVM: Use direct vcpu pointer during vCPU create/free [07/45] KVM: x86: Allocate vcpu struct in common x86 code [08/45] KVM: x86: Move FPU allocation to common x86 code [09/45] KVM: x86: Move allocation of pio_data page down a few lines [10/45] KVM: x86: Move kvm_vcpu_init() invocation to common code [11/45] KVM: PPC: e500mc: Add build-time assert that vcpu is at offset 0 [12/45] KVM: PPC: Allocate vcpu struct in common PPC code [13/45] KVM: PPC: Book3S PR: Allocate book3s and shadow vcpu after common init [14/45] KVM: PPC: e500mc: Move reset of oldpir below call to kvm_vcpu_init() [15/45] KVM: PPC: Move kvm_vcpu_init() invocation to common code [16/45] KVM: MIPS: Use kvm_vcpu_cache to allocate vCPUs [17/45] KVM: MIPS: Drop kvm_arch_vcpu_free() [18/45] KVM: PPC: Drop kvm_arch_vcpu_free() [19/45] KVM: arm: Drop kvm_arch_vcpu_free() [20/45] KVM: x86: Remove spurious kvm_mmu_unload() from vcpu destruction path [21/45] KVM: x86: Remove spurious clearing of async #PF MSR [22/45] KVM: x86: Drop kvm_arch_vcpu_free() [23/45] KVM: Remove kvm_arch_vcpu_free() declaration [24/45] KVM: Add kvm_arch_vcpu_precreate() to handle pre-allocation issues [25/45] KVM: s390: Move guts of kvm_arch_vcpu_init() into kvm_arch_vcpu_create() [26/45] KVM: s390: Invoke kvm_vcpu_init() before allocating sie_page [27/45] KVM: MIPS: Invoke kvm_vcpu_uninit() immediately prior to freeing vcpu [28/45] KVM: x86: Invoke kvm_vcpu_uninit() immediately prior to freeing vcpu [29/45] KVM: Introduce kvm_vcpu_destroy() [30/45] KVM: Move vcpu alloc and init invocation to common code [31/45] KVM: Unexport kvm_vcpu_cache and kvm_{un}init_vcpu() [32/45] KVM: Move initialization of preempt notifier to kvm_vcpu_init() [33/45] KVM: x86: Move guts of kvm_arch_vcpu_setup() into kvm_arch_vcpu_create() [34/45] KVM: MIPS: Move .vcpu_setup() call to kvm_arch_vcpu_create() [35/45] KVM: s390: Manually invoke vcpu setup during kvm_arch_vcpu_create() [36/45] KVM: PPC: BookE: Setup vcpu during kvmppc_core_vcpu_create() [37/45] KVM: Drop kvm_arch_vcpu_setup() [38/45] KVM: x86: Move all vcpu init code into kvm_arch_vcpu_create() [39/45] KVM: MIPS: Move all vcpu init code into kvm_arch_vcpu_create() [40/45] KVM: ARM: Move all vcpu init code into kvm_arch_vcpu_create() [41/45] KVM: PPC: Move all vcpu init code into kvm_arch_vcpu_create() [42/45] KVM: arm64: Free sve_state via arm specific hook [43/45] KVM: Drop kvm_arch_vcpu_init() and kvm_arch_vcpu_uninit() [44/45] KVM: Move putting of vcpu->pid to kvm_vcpu_destroy() [45/45] KVM: Move vcpu->run page allocation out of kvm_vcpu_init()

Commit Message

Sean Christopherson Oct. 22, 2019, 1:58 a.m. UTC

Do VPID allocation after calling the common kvm_vcpu_init() as a step
towards doing vCPU allocation (via kmem_cache_zalloc()) and calling
kvm_vcpu_init() back-to-back.  Squishing allocation and initialization
together will eventually allow the sequence to be moved to arch-agnostic
creation code.

Note, the VPID is not consumed until KVM_RUN, slightly delaying its
allocation should have no real function impact.  VPID allocation was
arbitrarily placed in the original patch, commit 2384d2b326408 ("KVM:
VMX: Enable Virtual Processor Identification (VPID)").

Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com>
---
 arch/x86/kvm/vmx/vmx.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff mbox series

Patch

diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c
index e660e28e9ae0..501e88bd6204 100644
--- a/arch/x86/kvm/vmx/vmx.c
+++ b/arch/x86/kvm/vmx/vmx.c
@@ -6726,14 +6726,14 @@  static struct kvm_vcpu *vmx_create_vcpu(struct kvm *kvm, unsigned int id)
 		goto free_user_fpu;
 	}
 
-	vmx->vpid = allocate_vpid();
-
 	err = kvm_vcpu_init(&vmx->vcpu, kvm, id);
 	if (err)
 		goto free_vcpu;
 
 	err = -ENOMEM;
 
+	vmx->vpid = allocate_vpid();
+
 	/*
 	 * If PML is turned on, failure on enabling PML just results in failure
 	 * of creating the vcpu, therefore we can simplify PML logic (by
@@ -6823,8 +6823,8 @@  static struct kvm_vcpu *vmx_create_vcpu(struct kvm *kvm, unsigned int id)
 	vmx_destroy_pml_buffer(vmx);
 uninit_vcpu:
 	kvm_vcpu_uninit(&vmx->vcpu);
+	free_vpid(vmx->vpid);
 free_vcpu:
-	free_vpid(vmx->vpid);
 	kmem_cache_free(x86_fpu_cache, vmx->vcpu.arch.guest_fpu);
 free_user_fpu:
 	kmem_cache_free(x86_fpu_cache, vmx->vcpu.arch.user_fpu);