Messages in this thread | | | Subject | Re: [PATCH v2 02/14] KVM: x86: simplify atomics in kvm_pit_ack_irq | From | Paolo Bonzini <> | Date | Fri, 19 Feb 2016 16:56:39 +0100 |
| |
On 19/02/2016 16:51, Radim Krčmář wrote: > The end result is going to be identical. I had a version that did > something similar and it was pretty tangled as well -- I wanted to > remove useless locks before re-using one for the ioctls. > (We need the protection earlier, because userspace can control notifiers > while PIT is still being initialized. And removing the lock had > dependencies.)
Yeah, I eventually imagined that cleaning up the locks helps with the patch that adds/removes the notifiers dynamically. Then I guess your current ordering of the patches is good!
Paolo
| |