On Fri, Apr 29, 2011 at 9:44 AM, Ingo Molnar <[email protected]> wrote: > Hm, this looks a bit strange (the mutex here protects only a kernel call - > that > cannot be right) and there's no explanation why it's needed. Why do > VIRTIO_BLK_IRQ (== KVM_IRQ_LINE ioctl()) calls have to be covered by the > mutex? > > A short blurb about expected behavior on SMP and locking rules at the top of > virtio-blk.c would be nice.
Yes, looks strange. Asias, did you see some bad behavior that this fixes? The per-device mutexes are there to protect device state. The assumption here is that KVM handles KVM_IRQ_LINE ioctl() serialization by titself. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
