Re: [PATCH 1/2] kvm tools: Fix IRQ assignments

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2011-05-06 at 10:43 +0300, Pekka Enberg wrote:
> On Fri, May 6, 2011 at 10:36 AM, Sasha Levin <levinsasha928@xxxxxxxxx> wrote:
> > afaik, we need to have same IRQ pins for devices which may have
> > different IRQ lines (All virtio-blk share same IRQ pin but different
> > line, I assume it'll be same with different virtio devices).
> >
> > So mptable has to manage association between device type and the
> > corresponding IRQ pin/line assignment, So dynamic assignment will be
> > more of a call with device type and return IRQ pin + line - which makes
> > it more complex than just allocating next free IRQ line.
> 
> Well then make kvm__request_irq() return a pointer to struct irq and
> have a hard-coded array of IRQ pin + line pairs in irq.c, for example.
> That'll reduce complexity in the virtio drivers and it'll serve as a
> starting point for proper IRQ allocator.

The big problem with dynamic allocations is having mptable register IRQ
sources properly, So I went ahead and deleted IRQ source definitions
from mptable.c and noticed that everything still works.

I've spoke with Cyrill and neither him or myself were too sure if it's
ok to do that now and if so, would it still be correct once SMP gets
added.

So here is my question: Would it be ok to drop IRQ source definitions
from mptable?

-- 

Sasha.

--
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


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux