Re: [Qemu-devel] [PATCHv2] Add KVM paravirt cpuid leaf

[Avi Kivity]

On 01/11/2010 09:18 PM, Anthony Liguori wrote:
> On 01/07/2010 10:24 AM, Gleb Natapov wrote:
> > diff --git a/target-i386/kvm.c b/target-i386/kvm.c
> > index 4084503..6a841de 100644
> > --- a/target-i386/kvm.c
> > +++ b/target-i386/kvm.c
> > @@ -17,6 +17,7 @@
> >   #include<sys/mman.h>
> >
> >   #include<linux/kvm.h>
> > +#include<linux/kvm_para.h>
>
> This breaks the build on a default F12 install because while kvm.h is 
> present, kvm_para.h is not.  This is a hard one to fix.
>
> We can default the kvm search path to /lib/modules/$(uname -r)/build, 
> we can fix the glibc headers and live with it, or we can pull in the 
> kvm headers into qemu.
>
> Avi/Marcelo/Jan, any thoughts from the qemu-kvm side?

Two options:

- make kvm detection depend on kvm_para.h being includable, fix all 
relevant distro kernels to supply it, and live with the breakage
- add a new CONFIG_KVM_PARA, detect it at configure time, and only 
include it if present

--
error compiling committee.c: too many arguments to function