>>> On Wed, Apr 4, 2007 at 10:20 AM, in message <[EMAIL PROTECTED]>,
Anthony Liguori <[EMAIL PROTECTED]> wrote: 
>
> The devices are already written to take a set_irq function.  Instead of 
> hijacking the emulated PIC device, I think it would be better if in 
> pc.c, we just conditionally created our PIC device that reflected to the 
> hypervisor and passed the appropriate function to the emulated hardware.
> 

When I first starting looking at the code, I was hoping to do exactly that.  
But unfortunately it appears as though many devices call pic_set_irq() directly 
which is why I used the approach I did.  Take a look at qemu/hw/rt18139.c for 
an example. 


> Otherwise, to support all the other architectures, there's going to be a 
> lot of modifications.

Fully agree, which is why I waited for review before putting in all that work ;)

-Greg

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Reply via email to