Avi Kivity wrote: > Zhang, Xiantao wrote: >> From 138335ed8c52e775a267a91086e580f86dd5b047 Mon Sep 17 00:00:00 >> 2001 >> From: Xiantao Zhang <xiantao.zhang@xxxxxxxxx> >> Date: Thu, 13 Nov 2008 15:23:31 +0800 >> Subject: [PATCH] KVM: Qemu: Sync compiler.h from kernel source when >> do make sync. >> >> In some OS distros, there is no compiler.h installed for building >> applications, and leads to break userspace build. >> Signed-off-by: Xiantao Zhang <xiantao.zhang@xxxxxxxxx> --- >> kernel/Makefile | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/kernel/Makefile b/kernel/Makefile >> index aca365e..ed4e2fd 100644 >> --- a/kernel/Makefile >> +++ b/kernel/Makefile >> @@ -64,6 +64,7 @@ header-sync: >> rm -rf $T >> rsync -R \ >> "$(LINUX)"/./include/linux/kvm*.h \ >> + "$(LINUX)"/./include/linux/compiler.h \ >> $(if $(wildcard $(headers-old)), $(headers-old)) \ >> $T/ $(if $(wildcard $(headers-new)), \ >> > > Maybe we should just remove the compiler.h include in > hack-modules.awk? I am not sure the inclusion for comiplier.h in <linux/kvm.h> is unnecessry when building userspace or external module. If it is useless, maybe we can remove it in hach. Could you confirm it ? Xiantao-- To unsubscribe from this list: send the line "unsubscribe kvm-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
