On Sun, 2008-03-23 at 16:19 +0000, Paul Brook wrote:
> On Sunday 23 March 2008, Dor Laor wrote:
> > --- a/qemu/hw/irq.c
> > +++ b/qemu/hw/irq.c
> > @@ -30,6 +30,8 @@ struct IRQState {
> > int n;
> > };
> >
> > +uint32_t qemu_irq_acked[NR_IRQ_WORDS];
>
> This is absolute rubbish. The whole point of the IRQ framework is that it
> doesn't assume a single flat IRQ controller.
>
Thanks for the compliments & the review ...
I specifically said that I'll move this variable into per-cpu var.
Moreover, the translation between irq line to vector is handled by the
'qemu_get_irq_vector' that calls 'irq_controller_get_vector' should take
care of the translation.
It works for ioapic, I'm not sure if it works for the flat pic case yet.
Anyway you're welcome to drift without the patch or provide constructive
comments.
> Paul
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel
