[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <49D27950.9020801@redhat.com>
Date: Tue, 31 Mar 2009 23:13:04 +0300
From: Avi Kivity <avi@...hat.com>
To: Gregory Haskins <ghaskins@...ell.com>
CC: linux-kernel@...r.kernel.org, agraf@...e.de, pmullaney@...ell.com,
pmorreale@...ell.com, anthony@...emonkey.ws, rusty@...tcorp.com.au,
netdev@...r.kernel.org, kvm@...r.kernel.org
Subject: Re: [RFC PATCH 15/17] kvm: add dynamic IRQ support
Gregory Haskins wrote:
>> - works with all guests
>> - supports hotplug/hotunplug, udev, sysfs, module autoloading, ...
>> - supported in all OSes
>> - someone else maintains it
>>
> These points are all valid, and I really struggled with this particular
> part of the design. The entire vbus design only requires one IRQ for
> the entire guest,
Won't this have scaling issues? One IRQ means one target vcpu. Whereas
I'd like virtio devices to span multiple queues, each queue with its own
MSI IRQ. Also, the single IRQ handler will need to scan for all
potential IRQ sources. Even if implemented carefully, this will cause
many cacheline bounces.
> so its conceivable that I could present a simple
> "dummy" PCI device with some "VBUS" type PCI-ID, just to piggy back on
> the IRQ routing logic. Then userspace could simply pass the IRQ routing
> info down to the kernel with an ioctl, or something similar.
>
Xen does something similar, I believe.
> I think ultimately I was trying to stay away from PCI in general because
> I want to support environments that do not have PCI. However, for the
> kvm-transport case (at least on x86) this isnt really a constraint.
>
>
s/PCI/the native IRQ solution for your platform/. virtio has the same
problem; on s390 we use the native (if that word ever applies to s390)
interrupt and device discovery mechanism.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists