KVM: arm/arm64: vgic: Move kvm_vgic_destroy call around
The way we call kvm_vgic_destroy is a bit bizarre. We call it *after* having freed the vcpus, which sort of defeats the point of cleaning up things before that point. Let's move kvm_vgic_destroy towards the beginning of kvm_arch_destroy_vm, which seems more sensible. Acked-by: Christoffer Dall <cdall@linaro.org> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com> Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
This commit is contained in:
parent
7129a9d8a6
commit
b2c9a85dd7
|
@ -177,6 +177,8 @@ void kvm_arch_destroy_vm(struct kvm *kvm)
|
||||||
{
|
{
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
|
kvm_vgic_destroy(kvm);
|
||||||
|
|
||||||
free_percpu(kvm->arch.last_vcpu_ran);
|
free_percpu(kvm->arch.last_vcpu_ran);
|
||||||
kvm->arch.last_vcpu_ran = NULL;
|
kvm->arch.last_vcpu_ran = NULL;
|
||||||
|
|
||||||
|
@ -186,8 +188,6 @@ void kvm_arch_destroy_vm(struct kvm *kvm)
|
||||||
kvm->vcpus[i] = NULL;
|
kvm->vcpus[i] = NULL;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
kvm_vgic_destroy(kvm);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
int kvm_vm_ioctl_check_extension(struct kvm *kvm, long ext)
|
int kvm_vm_ioctl_check_extension(struct kvm *kvm, long ext)
|
||||||
|
|
Loading…
Reference in New Issue