Re: [PATCH] kvm.h: __user requires compiler.h

[Avi Kivity <avi@qumranet.com>]

Anthony Liguori wrote:
> This patch breaks QEMU build when doing a 'make sync'.  When you do a 
> top-level ./configure, libkvm is built with kerneldir pointing to 
> kvm-userspace/kernel/include.  While linux/kvm.h is present there, 
> there isn't a linux/compiler.h.
>
> The host kernelpath isn't normally part of the libkvm or QEMU build.  
> So we have a couple options.
>
> 1) make the host kernelpath (/lib/modules/$(uname -r)/build/include) 
> part of the libkvm/QEMU build.
>
> 2) Do something else about __user
>
> Suggestions?  #1 might be a pain since there may be include conflicts 
> between the host kernel include and kernel/include.
>

We could hack 'make sync' to strip out __user (just like we run 
unifdef).  Of course the reasons for including linux/compiler.h are 
still valid, so it needs to remain.

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


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/