KVM: PPC: Book3S HV: Uninit vCPU if vcore creation fails
commit1a978d9d3e
upstream. Call kvm_vcpu_uninit() if vcore creation fails to avoid leaking any resources allocated by kvm_vcpu_init(), i.e. the vcpu->run page. Fixes:371fefd6f2
("KVM: PPC: Allow book3s_hv guests to use SMT processor modes") Cc: stable@vger.kernel.org Reviewed-by: Greg Kurz <groug@kaod.org> Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> Acked-by: Paul Mackerras <paulus@ozlabs.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
0718e2d3dc
commit
b2301deda8
|
@ -2354,7 +2354,7 @@ static struct kvm_vcpu *kvmppc_core_vcpu_create_hv(struct kvm *kvm,
|
||||||
mutex_unlock(&kvm->lock);
|
mutex_unlock(&kvm->lock);
|
||||||
|
|
||||||
if (!vcore)
|
if (!vcore)
|
||||||
goto free_vcpu;
|
goto uninit_vcpu;
|
||||||
|
|
||||||
spin_lock(&vcore->lock);
|
spin_lock(&vcore->lock);
|
||||||
++vcore->num_threads;
|
++vcore->num_threads;
|
||||||
|
@ -2371,6 +2371,8 @@ static struct kvm_vcpu *kvmppc_core_vcpu_create_hv(struct kvm *kvm,
|
||||||
|
|
||||||
return vcpu;
|
return vcpu;
|
||||||
|
|
||||||
|
uninit_vcpu:
|
||||||
|
kvm_vcpu_uninit(vcpu);
|
||||||
free_vcpu:
|
free_vcpu:
|
||||||
kmem_cache_free(kvm_vcpu_cache, vcpu);
|
kmem_cache_free(kvm_vcpu_cache, vcpu);
|
||||||
out:
|
out:
|
||||||
|
|
Loading…
Reference in New Issue