Hi Paolo,
On 06/25/2014 05:17 AM, Paolo Bonzini wrote:
Il 25/06/2014 11:28, James Hogan ha scritto:
On 24/06/14 18:31, Deng-Cheng Zhu wrote:
From: Deng-Cheng Zhu <dengcheng.zhu@xxxxxxxxxx>
kvm_arch_vcpu_free() is called in 2 code paths:
1) kvm_vm_ioctl()
kvm_vm_ioctl_create_vcpu()
kvm_arch_vcpu_destroy()
kvm_arch_vcpu_free()
2) kvm_put_kvm()
kvm_destroy_vm()
kvm_arch_destroy_vm()
kvm_mips_free_vcpus()
kvm_arch_vcpu_free()
Neither of the paths handles VCPU free. We need to do it in
kvm_arch_vcpu_free() corresponding to the memory allocation in
kvm_arch_vcpu_create().
Signed-off-by: Deng-Cheng Zhu <dengcheng.zhu@xxxxxxxxxx>
Reviewed-by: James Hogan <james.hogan@xxxxxxxxxx>
Maybe worth adding "Cc: stable@xxxxxxxxxxxxxxx" and moving this to the
beginning of the patchset to avoid conflicts.
Cheers
James
I've queued this for 3.16. It applies cleanly apart for the filename change.
Maybe you forgot to put this on the branch "queue"? I don't see it. Thanks.
Deng-Cheng
