On Wed, Jun 13, 2012 at 07:12:26PM +0300, Michael S. Tsirkin wrote: > > > > > @@ -134,10 +141,30 @@ int kvm_set_msi(struct kvm_kernel_irq_routing_entry *e, > > > > > irq.level = 1; > > > > > irq.shorthand = 0; > > > > > > > > > > + /* Multicast MSI doesn't really block but might take a long time. */ > > > > > + if (unlikely(noblock && kvm_msi_is_multicast(irq.dest_id, > > > > > + irq.delivery_mode))) > > > > delivery_mode? Should be dest_mode. > > Yes. Good catch, thanks. > > > But you probably need to check that > > > > delivery_mode is not ExtINT either. > > It does not look like anything happens with ExtInt > if you try to trigger it from MSI. > Currently no, but it should appear as if interrupt comes from PIC. I wouldn't allow anything but fixed mode here just to be on a safe side. Lowest prio will have to loop even after introducing irq cache. -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html