Hi, Avi

After Xiantao's irq_common patches were checked in, we found that it's
impossible to compile with VT-d userspace now. Essentially the problem is
Makefile missed a $ since unifdef patch checked in half an years ago...

But after I fix it, I found it's still impossible to get unifdef run
correctly...

First, unifdef report error when processing include/linux/kvm.h, but I
can't find out what's wrong now.

Second, seems at least my unifdef can't deal with

#if defined(CONFIG_X86) || defined(CONFIG_IA64)

My unifdef version is 1.0(20030701), the latest from debian testing. I also
tried one for fc9, same result.

How do you think...
--
regards
Yang,Sheng

--
From: Sheng Yang <[EMAIL PROTECTED]>
Date: Thu, 9 Oct 2008 20:45:02 +0800
Subject: [PATCH 1/1] kvm: Fix broken Makefile of kernel module


Signed-off-by: Sheng Yang <[EMAIL PROTECTED]>
---
 kernel/Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/Makefile b/kernel/Makefile
index f2a71fa..e352f77 100644
--- a/kernel/Makefile
+++ b/kernel/Makefile
@@ -65,7 +65,7 @@ header-sync:
             "$(LINUX)"/arch/$(ARCH_DIR)/include/asm/./kvm*.h \
              $T/include/asm-$(ARCH_DIR)/
 
-       set -e && for i in $(find $T -name '*.h'); do \
+       set -e && for i in $$(find $T -name '*.h'); do \
                $(call unifdef,$$i); done
        $(call hack, include/linux/kvm.h)
        set -e && for i in $$(find $T -type f -printf '%P '); \
@@ -79,7 +79,7 @@ source-sync:
             "$(LINUX)"/virt/kvm/./*.[cSh] \
             $T/
 
-       set -e && for i in $(find $T -name '*.c'); do \
+       set -e && for i in $$(find $T -name '*.c'); do \
                $(call unifdef,$$i); done
 
        for i in $(hack-files); \
-- 
1.5.3


--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to