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