Re: [PATCH v2] KVM: hyperv: fix locking of struct kvm_hv fields

2016-12-12 Thread Roman Kagan
On Mon, Dec 12, 2016 at 10:12:53AM +0100, Paolo Bonzini wrote: > Introduce a new mutex to avoid an AB-BA deadlock between kvm->lock and > vcpu->mutex. Protect accesses in kvm_hv_setup_tsc_page too, as suggested > by Roman. > > Reported-by: Dmitry Vyukov > Cc: Roman Kagan

Re: [PATCH v2] KVM: hyperv: fix locking of struct kvm_hv fields

2016-12-12 Thread Roman Kagan
On Mon, Dec 12, 2016 at 10:12:53AM +0100, Paolo Bonzini wrote: > Introduce a new mutex to avoid an AB-BA deadlock between kvm->lock and > vcpu->mutex. Protect accesses in kvm_hv_setup_tsc_page too, as suggested > by Roman. > > Reported-by: Dmitry Vyukov > Cc: Roman Kagan > Signed-off-by: Paolo

[PATCH v2] KVM: hyperv: fix locking of struct kvm_hv fields

2016-12-12 Thread Paolo Bonzini
Introduce a new mutex to avoid an AB-BA deadlock between kvm->lock and vcpu->mutex. Protect accesses in kvm_hv_setup_tsc_page too, as suggested by Roman. Reported-by: Dmitry Vyukov Cc: Roman Kagan Signed-off-by: Paolo Bonzini ---

[PATCH v2] KVM: hyperv: fix locking of struct kvm_hv fields

2016-12-12 Thread Paolo Bonzini
Introduce a new mutex to avoid an AB-BA deadlock between kvm->lock and vcpu->mutex. Protect accesses in kvm_hv_setup_tsc_page too, as suggested by Roman. Reported-by: Dmitry Vyukov Cc: Roman Kagan Signed-off-by: Paolo Bonzini --- Documentation/virtual/kvm/locking.txt | 8 ++--