[PATCH v2 0/6] kvm: level triggered irqfd support

2012-06-26 Thread Alex Williamson
Ok, let's see how this flies. I actually quite like this, so be gentle tearing it apart ;) I just couldn't bring myself to contort KVM_IRQFD into something that either sets up an irqfd or specifies a nearly unrelated EOI eventfd. The solution I've come up with, that also avoids exposing irq_sour

Re: [PATCH v2 0/6] kvm: level triggered irqfd support

2012-06-27 Thread Michael S. Tsirkin
On Tue, Jun 26, 2012 at 11:08:52PM -0600, Alex Williamson wrote: > I don't address whether injecting an edge irqfd really needs an assert > followed by de-assert (I don't know). So I just sent a patch removing that (works fine for me), and we'll see what others say. If it gets applied your patch w

Re: [PATCH v2 0/6] kvm: level triggered irqfd support

2012-06-27 Thread Michael S. Tsirkin
On Tue, Jun 26, 2012 at 11:08:52PM -0600, Alex Williamson wrote: > Ok, let's see how this flies. I actually quite like this, so be > gentle tearing it apart ;) > > I just couldn't bring myself to contort KVM_IRQFD into something > that either sets up an irqfd or specifies a nearly unrelated EOI >

Re: [PATCH v2 0/6] kvm: level triggered irqfd support

2012-06-27 Thread Alex Williamson
On Wed, 2012-06-27 at 12:58 +0300, Michael S. Tsirkin wrote: > On Tue, Jun 26, 2012 at 11:08:52PM -0600, Alex Williamson wrote: > > Ok, let's see how this flies. I actually quite like this, so be > > gentle tearing it apart ;) > > > > I just couldn't bring myself to contort KVM_IRQFD into somethi

Re: [PATCH v2 0/6] kvm: level triggered irqfd support

2012-06-28 Thread Michael S. Tsirkin
On Wed, Jun 27, 2012 at 08:33:40AM -0600, Alex Williamson wrote: > On Wed, 2012-06-27 at 12:58 +0300, Michael S. Tsirkin wrote: > > On Tue, Jun 26, 2012 at 11:08:52PM -0600, Alex Williamson wrote: > > > Ok, let's see how this flies. I actually quite like this, so be > > > gentle tearing it apart ;