Sam Ravnborg wrote:
>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
This patch also works.
--
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