From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] kvm.h: __user requires compiler.h Date: Tue, 25 Mar 2008 18:23:19 +0200 Message-ID: <47E926F7.6050503@qumranet.com> References: <200803121810.46055.borntraeger@de.ibm.com> <200803242046.17075.borntraeger@de.ibm.com> <47E80931.9000400@qumranet.com> <200803251642.42092.borntraeger@de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: kvm-devel@lists.sourceforge.net To: Christian Borntraeger Return-path: In-Reply-To: <200803251642.42092.borntraeger@de.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces@lists.sourceforge.net Errors-To: kvm-devel-bounces@lists.sourceforge.net List-Id: kvm.vger.kernel.org Christian Borntraeger wrote: > Am Montag, 24. M=E4rz 2008 schrieb Avi Kivity: > = >> Christian Borntraeger wrote: >> = >>> Am Freitag, 21. M=E4rz 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 :=3D 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 /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/