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

[Avi Kivity]

On 01/12/2010 09:23 AM, Gleb Natapov wrote:
> On Mon, Jan 11, 2010 at 01:18:32PM -0600, 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.
> >
> >      
> Avi how qemu-kvm compiles there? Or it doesn't?
>
>    

include/linux/Kbuild has:

ifneq ($(wildcard $(srctree)/arch/$(SRCARCH)/include/asm/kvm.h \
                  $(srctree)/include/asm-$(SRCARCH)/kvm.h),)
unifdef-y += kvm.h
endif
ifneq ($(wildcard $(srctree)/arch/$(SRCARCH)/include/asm/kvm_para.h \
                  $(srctree)/include/asm-$(SRCARCH)/kvm_para.h),)
unifdef-y += kvm_para.h
endif

So it should be installed.  Unfortunately this is starting 2.6.32, so we 
need to backport the patch (da18acffc3) to the F12 kernel.

qemu-kvm doesn't depend on installed headers.

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