Avi Kivity wrote: > Zhang, Xiantao wrote: >> Avi Kivity wrote: >> > >> Here, I don't think we can define io_apic.h under asm-ia64, since the >> similar definition exists in iosapic.h. Maybe we can removed the >> include here, and just define an common macro such kvm_dest_Fixed and >> kvm_dest_Low_Prio in irq.h to solve it, and make kvm not dependent of >> kernel's head files here. What's your opinion? >> > > Hm... I guess we can have a drivers/kvm/ioapic.h which will contain > the definitions for io[s]apic, and drop the dependency on > asm-x86/io_apic.h.
Hi Avi,
I didin't create the ioapic.h, since current irq.h has included
most the definitions of ioapic. Maybe we can create it later for irq.h
split. Now, I just put the definition in irq.h for delivery mode. Since
the delivery mode definitions doesn't depend on kernel header files, so
it will avoid possible failures with some special .configs.
Attached the patch.
Thanks
Xiantao
0003-Clearing-up-the-difference-of-ioapic-and-iosapic.patch
Description: 0003-Clearing-up-the-difference-of-ioapic-and-iosapic.patch
------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________ kvm-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kvm-devel
