slight annoyance with new x86 arch

slight annoyance with new x86 arch

[Boaz Harrosh]

Make system is a bit broken in the sense that it recompiles and links right after
a build even if nothing changed.

I have 
# CONFIG_LOCALVERSION_AUTO is not set

make output after a fully built tree:
{ // these are expected off course 
GEN /usr0/export/dev/bharrosh/git/pub/linux-2.6-block/.build_i386/Makefile
CHK include/linux/version.h
CHK include/linux/utsrelease.h
Using /usr0/export/dev/bharrosh/git/pub/linux-2.6-block as source for kernel
CALL /usr0/export/dev/bharrosh/git/pub/linux-2.6-block/scripts/checksyscalls.sh
CHK include/linux/compile.h
}
{ // these cause a recompilation and relink
AS arch/x86/kernel/vsyscall-int80_32.o
AS arch/x86/kernel/vsyscall-sysenter_32.o
SYSCALL arch/x86/kernel/vsyscall-syms.o
SYSCALL arch/x86/kernel/vsyscall-int80_32.so
SYSCALL arch/x86/kernel/vsyscall-sysenter_32.so
AS arch/x86/kernel/vsyscall_32.o
}

LD arch/x86/kernel/built-in.o
GEN .version
...
CC init/version.o
LD init/built-in.o
LD .tmp_vmlinux1
KSYM .tmp_kallsyms1.S
AS .tmp_kallsyms1.o
LD .tmp_vmlinux2
KSYM .tmp_kallsyms2.S
AS .tmp_kallsyms2.o
LD .tmp_vmlinux3
KSYM .tmp_kallsyms3.S
AS .tmp_kallsyms3.o
LD vmlinux.o
MODPOST vmlinux.o
....


I'm cross compiling on an x86_64 fedora7 machine. with
make ARCH=i386 KBUILD_OUTPUT=.build_i386 allmodconfig
and
make ARCH=i386 KBUILD_OUTPUT=.build_i386

Other wise after an mrproper all is well and booting
and the dir structure looks much better

Thanks
Boaz

[PATCH] x86: fix kernel rebuild due to vsyscall fallout

[Sam Ravnborg]

Fix rebuild of kernel when there is no changes.
This happened for i386.
Using make V=2 hinted that the output files were
not assigned to targets - fixed by this patch.

Reported by: Boaz Harrosh <bharrosh@panasas.com>

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
---
diff --git a/arch/x86/kernel/Makefile_32 b/arch/x86/kernel/Makefile_32
index 7ff0206..a3fa11f 100644
--- a/arch/x86/kernel/Makefile_32
+++ b/arch/x86/kernel/Makefile_32
@@ -51,7 +51,7 @@ obj-$(CONFIG_SCx200)		+= scx200_32.o
 # We must build both images before we can assemble it.
 # Note: kbuild does not track this dependency due to usage of .incbin
 $(obj)/vsyscall_32.o: $(obj)/vsyscall-int80_32.so $(obj)/vsyscall-sysenter_32.so
-targets += $(foreach F,int80 sysenter,vsyscall-$F.o vsyscall-$F.so)
+targets += $(foreach F,int80 sysenter,vsyscall-$F_32.o vsyscall-$F_32.so)
 targets += vsyscall-note_32.o vsyscall_32.lds
 
 # The DSO images are built using a special linker script.

Re: [PATCH] x86: fix kernel rebuild due to vsyscall fallout

[Thomas Gleixner]

On Wed, 17 Oct 2007, Sam Ravnborg wrote:

> Fix rebuild of kernel when there is no changes.
> This happened for i386.
> Using make V=2 hinted that the output files were
> not assigned to targets - fixed by this patch.
> 
> Reported by: Boaz Harrosh <bharrosh@panasas.com>
> 
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> ---
> diff --git a/arch/x86/kernel/Makefile_32 b/arch/x86/kernel/Makefile_32
> index 7ff0206..a3fa11f 100644
> --- a/arch/x86/kernel/Makefile_32
> +++ b/arch/x86/kernel/Makefile_32
> @@ -51,7 +51,7 @@ obj-$(CONFIG_SCx200)		+= scx200_32.o
>  # We must build both images before we can assemble it.
>  # Note: kbuild does not track this dependency due to usage of .incbin
>  $(obj)/vsyscall_32.o: $(obj)/vsyscall-int80_32.so $(obj)/vsyscall-sysenter_32.so
> -targets += $(foreach F,int80 sysenter,vsyscall-$F.o vsyscall-$F.so)
> +targets += $(foreach F,int80 sysenter,vsyscall-$F_32.o vsyscall-$F_32.so)
>  targets += vsyscall-note_32.o vsyscall_32.lds
>  
>  # The DSO images are built using a special linker script.

Thanks, applied

	tglx

 

[PATCH] Use relative symbolic links for bzImge in x86 ARCHs

[Boaz Harrosh]

OK One more problem with a patch this time

---
From: Benny Halevy <bhalevy@panasas.com>

use relative paths in the symlink to bzImage
to make it NFS export safe.

Signed-off-by: Benny Halevy <bhalevy@panasas.com>

diff --git a/arch/i386/Makefile b/arch/i386/Makefile
index f036d2d..396146e 100644
--- a/arch/i386/Makefile
+++ b/arch/i386/Makefile
@@ -132,7 +132,7 @@ zImage zlilo zdisk: KBUILD_IMAGE := arch/x86/boot/zImage
 
 zImage bzImage: vmlinux
 	$(Q)mkdir -p $(objtree)/arch/i386/boot
-	$(Q)ln -fsn $(objtree)/arch/x86/boot/bzImage $(objtree)/arch/i386/boot/bzImage
+	$(Q)ln -fsn ../../x86/boot/bzImage $(objtree)/arch/i386/boot/bzImage
 	$(Q)$(MAKE) $(build)=$(boot) $(KBUILD_IMAGE)
 
 compressed: zImage
diff --git a/arch/x86_64/Makefile b/arch/x86_64/Makefile
index 9daa32d..8048f4f 100644
--- a/arch/x86_64/Makefile
+++ b/arch/x86_64/Makefile
@@ -98,7 +98,7 @@ KBUILD_IMAGE                  := $(BOOTIMAGE)
 
 bzImage: vmlinux
 	$(Q)mkdir -p $(objtree)/arch/x86_64/boot
-	$(Q)ln -fsn $(objtree)/arch/x86/boot/bzImage $(objtree)/arch/x86_64/boot/bzImage
+	$(Q)ln -fsn ../../x86/boot/bzImage $(objtree)/arch/x86_64/boot/bzImage
 	$(Q)$(MAKE) $(build)=$(boot) $(BOOTIMAGE)
 
 bzlilo: vmlinux

Re: [PATCH] Use relative symbolic links for bzImge in x86 ARCHs

[Sam Ravnborg]

Hi Benny.

> use relative paths in the symlink to bzImage
> to make it NFS export safe.
> 
> Signed-off-by: Benny Halevy <bhalevy@panasas.com>

Your patch looks perfect - but you are a few hours too late.

See:
http://git.kernel.org/?p=linux/kernel/git/tglx/linux-2.6-x86.git;a=commit;h=a0075a509bd955ff6fc6e071efabb926e14bf19c

This is in the mm branch awaiting some more testing
before Thomas push it to Linus.

	Sam

Re: [PATCH] Use relative symbolic links for bzImge in x86 ARCHs

[Boaz Harrosh]

On Thu, Oct 18 2007 at 17:54 +0200, Sam Ravnborg <sam@ravnborg.org> wrote:
> Hi Benny.
> 
>> use relative paths in the symlink to bzImage
>> to make it NFS export safe.
>>
>> Signed-off-by: Benny Halevy <bhalevy@panasas.com>
> 
> Your patch looks perfect - but you are a few hours too late.
> 
> See:
> http://git.kernel.org/?p=linux/kernel/git/tglx/linux-2.6-x86.git;a=commit;h=a0075a509bd955ff6fc6e071efabb926e14bf19c
> 
> This is in the mm branch awaiting some more testing
> before Thomas push it to Linus.
> 
> 	Sam
NP,
as long as it's fixed I'm happy

Tested-by: Boaz Harrosh <bharrosh@panasas.com>

Thanks