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

[Avi Kivity <avi@qumranet.com>]

Christian Borntraeger wrote:
> Am Montag, 24. März 2008 schrieb Avi Kivity:
>   
>> Christian Borntraeger wrote:
>>     
>>> Am Freitag, 21. März 2008 schrieb Anthony Liguori:
>>>   
>>>       
>>>> 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,
>>>>         
> [...] 
>   
>> Maybe we should generate the 'make sync' headers using 'make 
>> headers_install'.
>>     
>
> headers_install works because there is 
>
> # Eliminate the contents of (and inclusions of) compiler.h
> HDRSED  := sed  -e "s/ inline / __inline__ /g" \
>                 -e "s/[[:space:]]__user[[:space:]]\{1,\}/ /g" \
>                 -e "s/(__user[[:space:]]\{1,\}/ (/g" \
>                 -e "s/[[:space:]]__force[[:space:]]\{1,\}/ /g" \
>                 -e "s/(__force[[:space:]]\{1,\}/ (/g" \
>                 -e "s/[[:space:]]__iomem[[:space:]]\{1,\}/ /g" \
>                 -e "s/(__iomem[[:space:]]\{1,\}/ (/g" \
>                 -e "s/[[:space:]]__attribute_const__[[:space:]]\{1,\}/\ /g" 
> \
>                 -e "s/[[:space:]]__attribute_const__$$//" \
>                 -e "/^\#include <linux\/compiler.h>/d"
>
> in scripts/Makefile.headersinst
>
> If you dont want to do something like for make sync, what about providing a 
> dummy compiler.h, which has
> only this line?
> #define __user 
>   

I already added something similar, see 
bcb30c8bafc7cac75c38981a57bc1f94521e83f4.


-- 
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/