>From e3b6fc4081669054768f93ea2605d1eba8e1b256 Mon Sep 17 00:00:00 2001 From: Sam Ravnborg <sam@xxxxxxxxxxxx> Date: Mon, 1 Dec 2008 19:41:49 +0100 Subject: [PATCH] sparc: fix boot regression Robert reported that 21cb8a43216e0595b466e903e0f2a7ae615d9e11 ("sparc: refactor Makefile") caused a boot regression on sparc. We tracked it down to the removal of the export of kallsyms.o. This patch adds back the export and the code that was removed in a subsequent commit which uses the exported variable. Why this caused a boot regression is not understood atm. Robert tested a simpler version of this patch and reported success. Reported-by: Robert Reif <reif@xxxxxxxxxxxxx> Tested-by: Robert Reif <reif@xxxxxxxxxxxxx> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx> --- The patch is made on top of sparc-next. Robert - I would appreciate if you could retest on top of sparc-next and confirm that the boot regression is really fixed there. I added a "Tested-by" since you tested the first version but would like you to really test this before Dave apply it. Note: Applying this will most likely cause a few trivial conflicts that I posted yesterday. But lets sort out this and then revisit that patch-set. Thanks, Sam arch/sparc/Makefile | 4 ++++ arch/sparc/boot/Makefile | 2 +- 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/arch/sparc/Makefile b/arch/sparc/Makefile index 6df8eb5..317aad6 100644 --- a/arch/sparc/Makefile +++ b/arch/sparc/Makefile @@ -89,6 +89,10 @@ VMLINUX_MAIN := $(core-y) kernel/ mm/ fs/ ipc/ security/ crypto/ block/ VMLINUX_MAIN += $(patsubst %/, %/lib.a, $(libs-y)) $(libs-y) VMLINUX_MAIN += $(drivers-y) $(net-y) +ifdef CONFIG_KALLSYMS +export kallsyms.o := .tmp_kallsyms2.o +endif + boot-y := arch/sparc/boot boot-$(CONFIG_SPARC64) := arch/sparc64/boot diff --git a/arch/sparc/boot/Makefile b/arch/sparc/boot/Makefile index 74ba65b..06b653d 100644 --- a/arch/sparc/boot/Makefile +++ b/arch/sparc/boot/Makefile @@ -42,7 +42,7 @@ BTOBJS := $(patsubst %/, %/built-in.o, $(VMLINUX_INIT)) BTLIBS := $(patsubst %/, %/built-in.o, $(VMLINUX_MAIN)) LDFLAGS_image := -T arch/sparc/kernel/vmlinux.lds $(BTOBJS) \ --start-group $(BTLIBS) --end-group \ - $(obj)/btfix.o + $(kallsyms.o) $(obj)/btfix.o # Link the final image including btfixup'ed symbols. # This is a replacement for the link done in the top-level Makefile. -- 1.5.6.GIT -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html