Mailed a new patch "kvm: fix usage of uninit spinlock in avic_vm_destroy()". On Tue, Jan 24, 2017 at 12:40 PM, Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > > > On 24/01/2017 12:34, Dmitry Vyukov wrote: >> On Tue, Jan 24, 2017 at 12:32 PM, Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: >>> >>> >>> On 23/01/2017 15:52, Dmitry Vyukov wrote: >>>> >>>> @@ -1382,6 +1383,9 @@ static void avic_vm_destroy(struct kvm *kvm) >>>> unsigned long flags; >>>> struct kvm_arch *vm_data = &kvm->arch; >>>> >>>> + if (!avic) >>>> + return 0; >>>> + >>>> avic_free_vm_id(vm_data->avic_vm_id); >>>> >>>> if (vm_data->avic_logical_id_table_page) >>> >>> Yes, this is the right one. >> >> >> I can mail this patch, but I can only test build. > > No problem, I have applied it locally and I am testing it. > > Paolo
