RE: [PATCH] KVM: Qemu: Sync compiler.h from kernel source when do
"Zhang, Xiantao" <[email protected]>
| Newsgroups | org.kernel.vger.kvm-ia64,org.kernel.vger.kvm |
|---|---|
| Message-ID | <706158FABBBA044BAD4FE898A02E4BC219A70E74@pdsmsx503.ccr.corp.intel.com> |
Avi Kivity wrote: > Zhang, Xiantao wrote: >> From 138335ed8c52e775a267a91086e580f86dd5b047 Mon Sep 17 00:00:00 >> 2001 >> From: Xiantao Zhang <[email protected]> >> 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 <[email protected]> --- >> 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