[coreboot] [commit] r5419 - in trunk/src/arch/i386: . init

repository service svn at coreboot.org
Tue Apr 13 15:43:35 CEST 2010


Author: stepan
Date: Tue Apr 13 15:43:35 2010
New Revision: 5419
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5419

Log:
clean up LD scripts and add some comments and proper license headers
where applicable.
Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>

Modified:
   trunk/src/arch/i386/coreboot_ram.ld
   trunk/src/arch/i386/init/ldscript_apc.lb
   trunk/src/arch/i386/init/ldscript_failover.lb
   trunk/src/arch/i386/init/ldscript_fallback_cbfs.lb

Modified: trunk/src/arch/i386/coreboot_ram.ld
==============================================================================
--- trunk/src/arch/i386/coreboot_ram.ld	Tue Apr 13 12:04:35 2010	(r5418)
+++ trunk/src/arch/i386/coreboot_ram.ld	Tue Apr 13 15:43:35 2010	(r5419)
@@ -1,11 +1,12 @@
 /*
  *	Memory map:
  *
- *	CONFIG_RAMBASE		
+ *	CONFIG_RAMBASE		: text segment
+ *				: rodata segment
  *				: data segment
  *				: bss segment
- *				: heap
  *				: stack
+ *				: heap
  */
 /*
  * Bootstrap code for the STPC Consumer
@@ -17,10 +18,8 @@
  *      Rewritten by Eric Biederman
  *  2005.12 yhlu add coreboot_ram cross the vga font buffer handling
  */
-/*
- *	We use ELF as output format. So that we can
- *	debug the code in some form. 
- */
+
+/* We use ELF as output format. So that we can debug the code in some form. */
 INCLUDE ldoptions
 
 ENTRY(_start)
@@ -28,9 +27,8 @@
 SECTIONS
 {
 	. = CONFIG_RAMBASE;
-	/*
-	 * First we place the code and read only data (typically const declared).
-	 * This get placed in rom.
+	/* First we place the code and read only data (typically const declared).
+	 * This could theoretically be placed in rom.
 	 */
 	.text : {
 		_text = .;
@@ -39,6 +37,7 @@
 		. = ALIGN(16);
 		_etext = .;
 	}
+
 	.rodata : {
 		_rodata = .;
 		. = ALIGN(4);
@@ -54,17 +53,14 @@
 		ecpu_drivers = . ;
 		*(.rodata)
 		*(.rodata.*)
-		/*
-		 * kevinh/Ispiri - Added an align, because the objcopy tool
+		/* kevinh/Ispiri - Added an align, because the objcopy tool
 		 * incorrectly converts sections that are not long word aligned.
-		 * This breaks the coreboot.rom target.
 		 */
 		 . = ALIGN(4);
 
 		_erodata = .;
 	}	
-	/*
-	 * After the code we place initialized data (typically initialized
+	/* After the code we place initialized data (typically initialized
 	 * global variables). This gets copied into ram by startup code.
 	 * __data_start and __data_end shows where in ram this should be placed,
 	 * whereas __data_loadstart and __data_loadend shows where in rom to
@@ -76,18 +72,7 @@
 		_edata = .;
 	}
 
-	.sdata : {
-		_SDA_BASE_ = .;
-		*(.sdata)
-	}
-
-	.sdata2 : {
-		_SDA2_BASE_ = .;
-		*(.sdata2)
-	}
-
-	/*
-	 * bss does not contain data, it is just a space that should be zero
+	/* bss does not contain data, it is just a space that should be zero
 	 * initialized on startup. (typically uninitialized global variables)
 	 * crt0.S fills between _bss and _ebss with zeroes.
 	 */
@@ -99,6 +84,11 @@
 	}
 	_ebss = .;
 	_end = .;
+
+	/* coreboot really "ends" here. Only heap and stack are placed after
+	 * this line.
+	 */
+
 	. = ALIGN(CONFIG_STACK_SIZE);
 
 	_stack = .;
@@ -107,6 +97,7 @@
 		. += CONFIG_MAX_CPUS*CONFIG_STACK_SIZE;
 	}
 	_estack = .;
+
         _heap = .;
         .heap . : {
                 /* Reserve CONFIG_HEAP_SIZE bytes for the heap */
@@ -115,18 +106,33 @@
         }
         _eheap = .;
 
+	/* Some assertions to print human readable errors for certain linker
+	 * error scenarios.
+	 */
+
 	/* Avoid running into 0xa0000-0xfffff */
 	_bogus = ASSERT(CONFIG_RAMBASE >= 0x100000 || _eheap < 0xa0000, "Please move RAMBASE to 1MB");
 
-	/* The ram segment
- 	 * This is all address of the memory resident copy of coreboot.
+	/* The ram segment. This includes all memory used by the memory 
+	 * resident copy of coreboot, except the tables that are produced on
+	 * the fly, but including stack and heap.
 	 */
 	_ram_seg = _text; 
 	_eram_seg = _eheap;
 
-	_bogus = ASSERT( ( _eram_seg < (CONFIG_RAMTOP)) , "please increase CONFIG_RAMTOP");
+	/* CONFIG_RAMTOP is the upper address of cached memory (among other
+	 * things). We must not exceed beyond that address, there be dragons.
+	 */
+	_bogus = ASSERT( ( _eram_seg < (CONFIG_RAMTOP)) , "Please increase CONFIG_RAMTOP");
+
+	/* This rule is only good for the few broken targets that still live
+	 * below 1MB per default. Those are the Geode and VIA targets that come
+	 * with their own version of real mode switches that can't live above
+	 * 1MB. Once these are fixed, this rule should go away.
+	 */
+        _bogus = ASSERT( !((CONFIG_CONSOLE_VGA || CONFIG_PCI_ROM_RUN || CONFIG_VGA_ROM_RUN || CONFIG_HAVE_SMI_HANDLER) && ((_ram_seg<0xa0000) && (_eram_seg>0xa0000))), "Please increase CONFIG_RAMTOP and if still fail, try to set CONFIG_RAMBASE to 1M");
 
-        _bogus = ASSERT( !((CONFIG_CONSOLE_VGA || CONFIG_PCI_ROM_RUN) && ((_ram_seg<0xa0000) && (_eram_seg>0xa0000))) , "please increase CONFIG_RAMTOP and if still fail, try to set CONFIG_RAMBASE more than 1M");
+	/* Discard the sections we don't need/want */
 
 	/DISCARD/ : {
 		*(.comment)

Modified: trunk/src/arch/i386/init/ldscript_apc.lb
==============================================================================
--- trunk/src/arch/i386/init/ldscript_apc.lb	Tue Apr 13 12:04:35 2010	(r5418)
+++ trunk/src/arch/i386/init/ldscript_apc.lb	Tue Apr 13 15:43:35 2010	(r5419)
@@ -1,4 +1,23 @@
-/* INPUT(coreboot_ap.rom)*/
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2006 Advanced Micro Devices, Inc.
+ * Copyright (C) 2008-2010 coresystems GmbH
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ */
+
 INCLUDE "ldoptions"
 SECTIONS
 {
@@ -9,8 +28,4 @@
 		*(.rodata.*)
                 _eapcrom = .;
         }
-        _iseg_apc = CONFIG_DCACHE_RAM_BASE;
-        _eiseg_apc = _iseg_apc + SIZEOF(.apcrom);
-        _liseg_apc = _apcrom;
-        _eliseg_apc = _eapcrom;
 }

Modified: trunk/src/arch/i386/init/ldscript_failover.lb
==============================================================================
--- trunk/src/arch/i386/init/ldscript_failover.lb	Tue Apr 13 12:04:35 2010	(r5418)
+++ trunk/src/arch/i386/init/ldscript_failover.lb	Tue Apr 13 15:43:35 2010	(r5419)
@@ -1,29 +1,24 @@
 /*
- *	Memory map:
+ * This file is part of the coreboot project.
  *
- *	CONFIG_RAMBASE		
- *				: data segment
- *				: bss segment
- *				: heap
- *				: stack
- *	CONFIG_ROMBASE
- *				: coreboot text 
- *				: readonly text
- */
-/*
- * Bootstrap code for the STPC Consumer
- * Copyright (c) 1999 by Net Insight AB. All Rights Reserved.
+ * Copyright (C) 2006 Advanced Micro Devices, Inc.
+ * Copyright (C) 2008-2010 coresystems GmbH
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
  *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-/*
- *	Written by Johan Rydberg, based on work by Daniel Kahlin.
- *      Rewritten by Eric Biederman
- */
-/*
- *	We use ELF as output format. So that we can
- *	debug the code in some form. 
- */
+/* We use ELF as output format. So that we can debug the code in some form. */
 OUTPUT_FORMAT("elf32-i386", "elf32-i386", "elf32-i386")
 OUTPUT_ARCH(i386)
 
@@ -34,8 +29,6 @@
 TARGET(binary)
 SECTIONS
 {
-	. = 0;
-
 	/* This section might be better named .setup */
 	.rom ROMLOC : {
 		_rom = .;
@@ -44,7 +37,7 @@
 		*(.rom.data.*);
 		*(.rodata.*);
 		_erom = .;
-	} >rom =0xff
+	} >rom = 0xff
 
 	ROMLOC = 0xffffff00 - (_erom - _rom) + 1;
 

Modified: trunk/src/arch/i386/init/ldscript_fallback_cbfs.lb
==============================================================================
--- trunk/src/arch/i386/init/ldscript_fallback_cbfs.lb	Tue Apr 13 12:04:35 2010	(r5418)
+++ trunk/src/arch/i386/init/ldscript_fallback_cbfs.lb	Tue Apr 13 15:43:35 2010	(r5419)
@@ -1,36 +1,27 @@
 /*
- *	Memory map:
+ * This file is part of the coreboot project.
  *
- *	CONFIG_RAMBASE		
- *				: data segment
- *				: bss segment
- *				: heap
- *				: stack
- *	CONFIG_ROMBASE
- *				: coreboot text 
- *				: readonly text
- */
-/*
- * Bootstrap code for the STPC Consumer
- * Copyright (c) 1999 by Net Insight AB. All Rights Reserved.
+ * Copyright (C) 2006 Advanced Micro Devices, Inc.
+ * Copyright (C) 2008-2010 coresystems GmbH
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
  *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-/*
- *	Written by Johan Rydberg, based on work by Daniel Kahlin.
- *      Rewritten by Eric Biederman
- */
-/*
- *	We use ELF as output format. So that we can
- *	debug the code in some form. 
- */
+/* We use ELF as output format. So that we can debug the code in some form. */
 OUTPUT_FORMAT("elf32-i386", "elf32-i386", "elf32-i386")
 OUTPUT_ARCH(i386)
 
-/*
-ENTRY(_start)
-*/
-
 TARGET(binary)
 SECTIONS
 {
@@ -45,8 +36,6 @@
 		_rom = .;
 		*(.rom.text);
 		*(.rom.data);
-		*(.init.rodata.*);
-		*(.init.text);
 		*(.rodata);
 		*(.rodata.*);
 		*(.rom.data.*);




More information about the coreboot mailing list