Received: from mail.onstor.com ([66.201.51.107]) by onstor-exch02.onstor.net with Microsoft SMTPSVC(6.0.3790.1830);
	 Sat, 15 Sep 2007 14:35:18 -0700
Received: from ftp.linux-mips.org ([194.74.144.162]) by mail.onstor.com with Microsoft SMTPSVC(6.0.3790.1830);
	 Sat, 15 Sep 2007 14:35:18 -0700
Received: from localhost.localdomain ([127.0.0.1]:51079 "EHLO
	ftp.linux-mips.org") by ftp.linux-mips.org with ESMTP
	id S20022325AbXIOVe7 convert rfc822-to-quoted-printable (ORCPT
	<rfc822;dan.stein@onstor.com> + 1 other);
	Sat, 15 Sep 2007 22:34:59 +0100
Received: with ECARTIS (v1.0.0; list linux-mips); Sat, 15 Sep 2007 22:34:39 +0100 (BST)
Received: from pasmtpb.tele.dk ([80.160.77.98]:48047 "EHLO pasmtpB.tele.dk")
	by ftp.linux-mips.org with ESMTP id S20022322AbXIOVea (ORCPT
	<rfc822;linux-mips@linux-mips.org>); Sat, 15 Sep 2007 22:34:30 +0100
Received: from ravnborg.org (0x535d98d8.vgnxx8.adsl-dhcp.tele.dk [83.93.152.216])
	by pasmtpB.tele.dk (Postfix) with ESMTP id A1320E3251B;
	Sat, 15 Sep 2007 23:34:27 +0200 (CEST)
Received: by ravnborg.org (Postfix, from userid 1000)
	id 979A7580D2; Sat, 15 Sep 2007 23:35:53 +0200 (CEST)
Date:	Sat, 15 Sep 2007 23:35:53 +0200
From:	Sam Ravnborg <sam@ravnborg.org>
To:	ralf@linux-mips.org
Cc:	linux-mips@linux-mips.org
Subject: [PATCH] mips: beautify vmlinux.lds
Message-ID: <20070915213553.GA15463@uranus.ravnborg.org>
Mime-Version: 1.0
Content-Type:	text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: QUOTED-PRINTABLE
User-Agent: Mutt/1.4.2.1i
X-archive-position: 16530
X-ecartis-version: Ecartis v1.0.0
Sender:	linux-mips-bounce@linux-mips.org
Errors-to: linux-mips-bounce@linux-mips.org
X-original-sender: sam@ravnborg.org
Precedence: bulk
X-list:	linux-mips
Return-Path: linux-mips-bounce@linux-mips.org
X-OriginalArrivalTime: 15 Sep 2007 21:35:18.0100 (UTC) FILETIME=[4F72B140:01C7F7E0]

Introduce a consistent style for vmlinux.lds.
This style will be consitent with all other arch's - soon.

In addition:
- Moved a few labels inside brackets for the sections they specify
  to prevent that linker alignmnet made them point before the section s=
tart

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
---
Patch is not even buildtested due to lack of a toolchain :-(

	Sam

diff --git a/arch/mips/kernel/vmlinux.lds.S b/arch/mips/kernel/vmlinux.=
lds.S
index 60bbaec..84f9a4c 100644
--- a/arch/mips/kernel/vmlinux.lds.S
+++ b/arch/mips/kernel/vmlinux.lds.S
@@ -6,161 +6,202 @@
 OUTPUT_ARCH(mips)
 ENTRY(kernel_entry)
 jiffies =3D JIFFIES;
+
 SECTIONS
 {
 #ifdef CONFIG_BOOT_ELF64
-  /* Read-only sections, merged into text segment: */
-  /* . =3D 0xc000000000000000; */
+	/* Read-only sections, merged into text segment: */
+	/* . =3D 0xc000000000000000; */
=20
-  /* This is the value for an Origin kernel, taken from an IRIX kernel=
=2E  */
-  /* . =3D 0xc00000000001c000; */
+	/* This is the value for an Origin kernel, taken from an IRIX kernel.=
  */
+	/* . =3D 0xc00000000001c000; */
=20
-  /* Set the vaddr for the text segment to a value
-        >=3D 0xa800 0000 0001 9000 if no symmon is going to configured
-        >=3D 0xa800 0000 0030 0000 otherwise  */
+	/* Set the vaddr for the text segment to a value
+	 *   >=3D 0xa800 0000 0001 9000 if no symmon is going to configured
+	 *   >=3D 0xa800 0000 0030 0000 otherwise
+	 */
=20
-  /* . =3D 0xa800000000300000; */
-  /* . =3D 0xa800000000300000; */
-  . =3D 0xffffffff80300000;
+	/* . =3D 0xa800000000300000; */
+	/* . =3D 0xa800000000300000; */
+	. =3D 0xffffffff80300000;
 #endif
-  . =3D LOADADDR;
-  /* read-only */
-  _text =3D .;			/* Text and read-only data */
-  .text : {
-    TEXT_TEXT
-    SCHED_TEXT
-    LOCK_TEXT
-    *(.fixup)
-    *(.gnu.warning)
-  } =3D0
-
-  _etext =3D .;			/* End of text section */
-
-  . =3D ALIGN(16);		/* Exception table */
-  __start___ex_table =3D .;
-  __ex_table : { *(__ex_table) }
-  __stop___ex_table =3D .;
-
-  __start___dbe_table =3D .;	/* Exception table for data bus errors */
-  __dbe_table : { *(__dbe_table) }
-  __stop___dbe_table =3D .;
-
-  RODATA
-
-  /* writeable */
-  .data : {			/* Data */
-    . =3D . + DATAOFFSET;		/* for CONFIG_MAPPED_KERNEL */
-    /*
-     * This ALIGN is needed as a workaround for a bug a gcc bug upto 4=
=2E1 which
-     * limits the maximum alignment to at most 32kB and results in the=
 following
-     * warning:
-     *
-     *  CC      arch/mips/kernel/init_task.o
-     * arch/mips/kernel/init_task.c:30: warning: alignment of =E2=80=98=
init_thread_union=E2=80=99
-     * is greater than maximum object file alignment.  Using 32768
-     */
-    . =3D ALIGN(_PAGE_SIZE);
-    *(.data.init_task)
-
-    DATA_DATA
-
-    CONSTRUCTORS
-  }
-  _gp =3D . + 0x8000;
-  .lit8 : { *(.lit8) }
-  .lit4 : { *(.lit4) }
-  /* We want the small data sections together, so single-instruction o=
ffsets
-     can access them all, and initialized data all before uninitialize=
d, so
-     we can shorten the on-disk segment size.  */
-  .sdata     : { *(.sdata) }
-
-  . =3D ALIGN(_PAGE_SIZE);
-  __nosave_begin =3D .;
-  .data_nosave : { *(.data.nosave) }
-  . =3D ALIGN(_PAGE_SIZE);
-  __nosave_end =3D .;
-
-  . =3D ALIGN(32);
-  .data.cacheline_aligned : { *(.data.cacheline_aligned) }
-
-  _edata =3D  .;			/* End of data section */
-
-  /* will be freed after init */
-  . =3D ALIGN(_PAGE_SIZE);		/* Init code and data */
-  __init_begin =3D .;
-  .init.text : {
-	_sinittext =3D .;
-	*(.init.text)
-	_einittext =3D .;
-  }
-  .init.data : { *(.init.data) }
-  . =3D ALIGN(16);
-  __setup_start =3D .;
-  .init.setup : { *(.init.setup) }
-  __setup_end =3D .;
-
-  __initcall_start =3D .;
-  .initcall.init : {
-	INITCALLS
-  }
-  __initcall_end =3D .;
-
-  __con_initcall_start =3D .;
-  .con_initcall.init : { *(.con_initcall.init) }
-  __con_initcall_end =3D .;
-  SECURITY_INIT
-    /* .exit.text is discarded at runtime, not link time, to deal with
-     references from .rodata */
-  .exit.text : { *(.exit.text) }
-  .exit.data : { *(.exit.data) }
+	. =3D LOADADDR;
+	/* read-only */
+	_text =3D .;	/* Text and read-only data */
+	.text : {
+		TEXT_TEXT
+		SCHED_TEXT
+		LOCK_TEXT
+		*(.fixup)
+		*(.gnu.warning)
+	} =3D0
+	_etext =3D .;	/* End of text section */
+
+	/* Exception table */
+	. =3D ALIGN(16);
+	__ex_table : {
+		__start___ex_table =3D .;
+		*(__ex_table)
+		__stop___ex_table =3D .;
+	}
+
+	/* Exception table for data bus errors */
+	__dbe_table : {
+		__start___dbe_table =3D .;
+		*(__dbe_table)
+		__stop___dbe_table =3D .;
+	}
+	RODATA
+
+	/* writeable */
+	.data : {	/* Data */
+	  . =3D . + DATAOFFSET;		/* for CONFIG_MAPPED_KERNEL */
+	  /*
+	   * This ALIGN is needed as a workaround for a bug a gcc bug upto 4.=
1 which
+	   * limits the maximum alignment to at most 32kB and results in the =
following
+	   * warning:
+	   *
+	   *  CC      arch/mips/kernel/init_task.o
+	   * arch/mips/kernel/init_task.c:30: warning: alignment of =E2=80=98=
init_thread_union=E2=80=99
+	   * is greater than maximum object file alignment.  Using 32768
+	   */
+	  . =3D ALIGN(_PAGE_SIZE);
+	  *(.data.init_task)
+
+	  DATA_DATA
+	  CONSTRUCTORS
+	}
+	_gp =3D . + 0x8000;
+	.lit8 : {
+		*(.lit8)
+	}
+	.lit4 : {
+		*(.lit4)
+	}
+	/* We want the small data sections together, so single-instruction of=
fsets
+	   can access them all, and initialized data all before uninitialized=
, so
+	   we can shorten the on-disk segment size.  */
+	.sdata : {
+		*(.sdata)
+	}
+
+	. =3D ALIGN(_PAGE_SIZE);
+	.data_nosave : {
+		__nosave_begin =3D .;
+		*(.data.nosave)
+	}
+	. =3D ALIGN(_PAGE_SIZE);
+	__nosave_end =3D .;
+
+	. =3D ALIGN(32);
+	.data.cacheline_aligned : {
+		*(.data.cacheline_aligned)
+	}
+	_edata =3D  .;			/* End of data section */
+
+	/* will be freed after init */
+	. =3D ALIGN(_PAGE_SIZE);		/* Init code and data */
+	__init_begin =3D .;
+	.init.text : {
+		_sinittext =3D .;
+		*(.init.text)
+		_einittext =3D .;
+	}
+	.init.data : {
+		*(.init.data)
+	}
+	. =3D ALIGN(16);
+	.init.setup : {
+		__setup_start =3D .;
+		*(.init.setup)
+		__setup_end =3D .;
+	}
+
+	.initcall.init : {
+		__initcall_start =3D .;
+		INITCALLS
+		__initcall_end =3D .;
+	}
+
+	.con_initcall.init : {
+		__con_initcall_start =3D .;
+		*(.con_initcall.init)
+		__con_initcall_end =3D .;
+	}
+	SECURITY_INIT
+
+	/* .exit.text is discarded at runtime, not link time, to deal with
+	 * references from .rodata
+	 */
+	.exit.text : {
+		*(.exit.text)
+	}
+	.exit.data : {
+		*(.exit.data)
+	}
 #if defined(CONFIG_BLK_DEV_INITRD)
-  . =3D ALIGN(_PAGE_SIZE);
-  __initramfs_start =3D .;
-  .init.ramfs : { *(.init.ramfs) }
-  __initramfs_end =3D .;
+	. =3D ALIGN(_PAGE_SIZE);
+	.init.ramfs : {
+		__initramfs_start =3D .;
+		*(.init.ramfs)
+		__initramfs_end =3D .;
+	}
 #endif
-  PERCPU(_PAGE_SIZE)
-  . =3D ALIGN(_PAGE_SIZE);
-  __init_end =3D .;
-  /* freed after init ends here */
-
-  __bss_start =3D .;		/* BSS */
-  .sbss      : {
-    *(.sbss)
-    *(.scommon)
-  }
-  .bss : {
-    *(.bss)
-    *(COMMON)
-  }
-  __bss_stop =3D .;
-
-  _end =3D . ;
-
-  /* Sections to be discarded */
-  /DISCARD/ : {
-        *(.exitcall.exit)
-
-	/* ABI crap starts here */
-	*(.MIPS.options)
-	*(.options)
-	*(.pdr)
-	*(.reginfo)
-  }
-
-  /* These mark the ABI of the kernel for debuggers.  */
-  .mdebug.abi32 : { KEEP(*(.mdebug.abi32)) }
-  .mdebug.abi64 : { KEEP(*(.mdebug.abi64)) }
-
-  /* This is the MIPS specific mdebug section.  */
-  .mdebug : { *(.mdebug) }
-
-  STABS_DEBUG
-
-  DWARF_DEBUG
-
-  /* These must appear regardless of  .  */
-  .gptab.sdata : { *(.gptab.data) *(.gptab.sdata) }
-  .gptab.sbss : { *(.gptab.bss) *(.gptab.sbss) }
-  .note : { *(.note) }
+	PERCPU(_PAGE_SIZE)
+	. =3D ALIGN(_PAGE_SIZE);
+	__init_end =3D .;
+	/* freed after init ends here */
+
+	__bss_start =3D .;	/* BSS */
+	.sbss  : {
+		*(.sbss)
+		*(.scommon)
+	}
+	.bss : {
+		*(.bss)
+		*(COMMON)
+	}
+	__bss_stop =3D .;
+
+	_end =3D . ;
+
+	/* Sections to be discarded */
+	/DISCARD/ : {
+		*(.exitcall.exit)
+
+		/* ABI crap starts here */
+		*(.MIPS.options)
+		*(.options)
+		*(.pdr)
+		*(.reginfo)
+	}
+
+	/* These mark the ABI of the kernel for debuggers.  */
+	.mdebug.abi32 : {
+		KEEP(*(.mdebug.abi32))
+	}
+	.mdebug.abi64 : {
+		KEEP(*(.mdebug.abi64))
+	}
+
+	/* This is the MIPS specific mdebug section.  */
+	.mdebug : {
+		*(.mdebug)
+	}
+
+	STABS_DEBUG
+	DWARF_DEBUG
+
+	/* These must appear regardless of  .  */
+	.gptab.sdata : {
+		*(.gptab.data)
+		*(.gptab.sdata)
+	}
+	.gptab.sbss : {
+		*(.gptab.bss)
+		*(.gptab.sbss)
+	}
+	.note : {
+		*(.note)
+	}
 }

