Re: [PATCH v2 9/9] KVM: mark requests that need synchronization

2017-04-27 Thread Paolo Bonzini
On 26/04/2017 22:32, Radim Krčmář wrote: > v2: replaces [v1 1/6] > Ugh, KVM_ARCH_REQ_WAIT_NO_WAKEUP looks a weird ... Yeah, let's drop patch 7 and just use bits for now. I think using KVM_ARCH_REQ_FLAGS directly should be fine, especially after the default is flipped from "no wakeup" to "

Re: [PATCH v2 9/9] KVM: mark requests that need synchronization

2017-04-27 Thread Andrew Jones
On Wed, Apr 26, 2017 at 10:32:27PM +0200, Radim Krčmář wrote: > kvm_make_all_requests() provides a synchronization that waits until all > kicked VCPUs have acknowledged the kick. This is important for > KVM_REQ_MMU_RELOAD as it prevents freeing while lockless paging is > underway. > > This patch

[PATCH v2 9/9] KVM: mark requests that need synchronization

2017-04-26 Thread Radim Krčmář
kvm_make_all_requests() provides a synchronization that waits until all kicked VCPUs have acknowledged the kick. This is important for KVM_REQ_MMU_RELOAD as it prevents freeing while lockless paging is underway. This patch adds the synchronization property into all requests that are currently bei