On Tue, Feb 22, 2011 at 09:47:09PM +0100, Linus Torvalds wrote: > On Tue, Feb 22, 2011 at 9:08 AM, Jesper Nilsson <[email protected]> > wrote: > > True, at least until I've made sure that there isn't any > > underlying reason for CRISv32 to put __param in a different place... > > > > Acked-by: Jesper Nilsson <[email protected]> > > I really am not going to apply a patch that likely just adds another > redundant field to the linker scripts. So the ack I'm just ignoring,
:-) > But it would be good to get the "remove the legacy __param thing > because it's already there from the asm-generic version" confirmed by > testing, and not just looking at the source code. Testing shows no problems with the below patch. >From f8c17a94940a012f237986b4f40ca557a88f9019 Mon Sep 17 00:00:00 2001 From: Jesper Nilsson <[email protected]> Date: Wed, 23 Feb 2011 13:04:25 +0100 Subject: [PATCH] Drop redundant __param section for CRISv32. The __param section is already brought in by RODATA above. Signed-off-by: Jesper Nilsson <[email protected]> --- arch/cris/kernel/vmlinux.lds.S | 5 ----- 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/arch/cris/kernel/vmlinux.lds.S b/arch/cris/kernel/vmlinux.lds.S index 917b727..e8adc35 100644 --- a/arch/cris/kernel/vmlinux.lds.S +++ b/arch/cris/kernel/vmlinux.lds.S @@ -75,11 +75,6 @@ SECTIONS INIT_TEXT_SECTION(PAGE_SIZE) .init.data : { INIT_DATA } .init.setup : { INIT_SETUP(16) } -#ifdef CONFIG_ETRAX_ARCH_V32 - __start___param = .; - __param : { *(__param) } - __stop___param = .; -#endif .initcall.init : { INIT_CALLS } -- 1.7.4 > Linus /^JN - Jesper Nilsson -- Jesper Nilsson -- [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
