Re: [Qemu-devel] [PATCH] Makefile: Fix compilation for non-standard host kernel path

[Anthony Liguori <anthony@codemonkey.ws>]

On 07/02/2010 06:15 AM, Prerna Saxena wrote:
> Set up host kernel include paths specified by --kerneldir
>
> When host kernel headers are placed in non-standard paths, the
> KVM_CFLAGS are presently invoked only for a few .c files 
> (kvm*.c,vhost*.c) and not for other files like machine.c, cpus.c
>  ..etc which also depend on linux/kvm.h

I think that's a bug.  What's the dependency of machine.c and cpus.c?

Regards,

Anthony Liguori

>
> Signed-off-by: Prerna Saxena <prerna@linux.vnet.ibm.com>
> ---
>  Makefile.target |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/Makefile.target b/Makefile.target
> index d58b201..b433112 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -29,12 +29,15 @@ QEMU_PROG=qemu-system-$(TARGET_ARCH2)$(EXESUF)
>  endif
>  endif
>
> +# Set up host kernel include paths specified by --kerneldir
> +ifdef CONFIG_KVM
> +QEMU_CFLAGS+=$(KVM_CFLAGS)
> +endif
> +
>  PROGS=$(QEMU_PROG)
>
>  LIBS+=-lm
>
> -kvm.o kvm-all.o vhost.o vhost_net.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
> -
>  config-target.h: config-target.h-timestamp
>  config-target.h-timestamp: config-target.mak
>