[PATCH] kvm: external module: Remove 'include/asm' before execute 'ln'

[PATCH] kvm: external module: Remove 'include/asm' before execute 'ln'

[Yang, Sheng]

[-- Attachment #1: Type: text/plain, Size: 1012 bytes --]

From 33d84813131809e962a4cd8cae9b66a9062fa8db Mon Sep 17 00:00:00 2001
From: Sheng Yang <sheng.yang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Date: Thu, 20 Dec 2007 14:22:18 +0800
Subject: [PATCH] kvm: external module: Remove 'include/asm' before 
execute 'ln'

We use 'ln -sf' to refresh the symbolic now. But if 'include/asm' is a 
directory (as in old layout), 'ln' will give wrong result.

Signed-off-by: Sheng Yang <sheng.yang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
 kernel/Makefile |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/kernel/Makefile b/kernel/Makefile
index d409a0a..e5d9fff 100644
--- a/kernel/Makefile
+++ b/kernel/Makefile
@@ -35,7 +35,8 @@ sync:
 	     "$(LINUX)"/./include/linux/kvm*.h \
 	     "$(LINUX)"/./include/asm-x86/kvm*.h \
              tmp/
-	ln -sf asm-x86 include/asm
+	rm -rf include/asm
+	ln -s asm-x86 include/asm
 
 	$(call unifdef, include/linux/kvm.h)
 	$(call unifdef, include/linux/kvm_para.h)
-- 
1.5.3.4

[-- Attachment #2: 0001-kvm-external-module-Remove-include-asm-before-ex.patch --]
[-- Type: text/x-diff, Size: 1011 bytes --]

From 33d84813131809e962a4cd8cae9b66a9062fa8db Mon Sep 17 00:00:00 2001
From: Sheng Yang <sheng.yang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Date: Thu, 20 Dec 2007 14:22:18 +0800
Subject: [PATCH] kvm: external module: Remove 'include/asm' before execute 'ln'

We use 'ln -sf' to refresh the symbolic now. But if 'include/asm' is a directory
(as in old layout), 'ln' will give wrong result.

Signed-off-by: Sheng Yang <sheng.yang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
 kernel/Makefile |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/kernel/Makefile b/kernel/Makefile
index d409a0a..e5d9fff 100644
--- a/kernel/Makefile
+++ b/kernel/Makefile
@@ -35,7 +35,8 @@ sync:
 	     "$(LINUX)"/./include/linux/kvm*.h \
 	     "$(LINUX)"/./include/asm-x86/kvm*.h \
              tmp/
-	ln -sf asm-x86 include/asm
+	rm -rf include/asm
+	ln -s asm-x86 include/asm
 
 	$(call unifdef, include/linux/kvm.h)
 	$(call unifdef, include/linux/kvm_para.h)
-- 
1.5.3.4


[-- Attachment #3: Type: text/plain, Size: 308 bytes --]

-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Re: [PATCH] kvm: external module: Remove 'include/asm' before execute 'ln'

[Avi Kivity]

Yang, Sheng wrote:
> From 33d84813131809e962a4cd8cae9b66a9062fa8db Mon Sep 17 00:00:00 2001
> From: Sheng Yang <sheng.yang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> Date: Thu, 20 Dec 2007 14:22:18 +0800
> Subject: [PATCH] kvm: external module: Remove 'include/asm' before 
> execute 'ln'
>
> We use 'ln -sf' to refresh the symbolic now. But if 'include/asm' is a 
> directory (as in old layout), 'ln' will give wrong result.
>
>   

Applied, thanks.  This has bitten several people already.


-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace