[Skiboot] [PATCH 2/2] With new GCC comes larger GCOV binaries

Stewart Smith stewart at linux.ibm.com
Tue May 21 11:45:38 AEST 2019


So we need to change our heap size to make more room for data/bss
without having to change where the console is or have more fun moving
things about.

Signed-off-by: Stewart Smith <stewart at linux.ibm.com>
---
 include/mem-map.h | 16 +++++++++-------
 skiboot.lds.S     |  6 +++---
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/include/mem-map.h b/include/mem-map.h
index 3e30d9b76111..fa5cacb1c610 100644
--- a/include/mem-map.h
+++ b/include/mem-map.h
@@ -59,8 +59,8 @@
  */
 #define CPU_CTL_OFF             (SPIRA_OFF + 0x1800)
 
-/* We keep a gap of 2M for skiboot text & bss for now. We will
- * then we have our heap which goes up to base + 14M (so 12M for
+/* We keep a gap of 5M for skiboot text & bss for now. We will
+ * then we have our heap which goes up to base + 14M (so 11M for
  * now, though we can certainly reduce that a lot).
  *
  * Ideally, we should fix the heap end and use _end to basically
@@ -71,14 +71,16 @@
  * regions statically in mem_region.c, but still on the list of
  * things to improve.
  *
- * As of this writing (2014/4/6), we use approc 512K for skiboot
+ * As of A Long Time Ago (2014/4/6), we used approc 512K for skiboot
  * core and 2M of heap on a 1 socket machine.
  *
- * As of 2015/5/7 we use approx 800k for skiboot, 500k HEAP for
- * mambo boot.
+ * As of still a Long Time Ago (2015/5/7) we used approx 800k for skiboot,
+ * 500k HEAP for mambo boot.
+ *
+ * As of mid-2019, a 2 socket Romulus uses ~4MB heap.
  */
-#define HEAP_BASE		(SKIBOOT_BASE + 0x00400000)
-#define HEAP_SIZE		0x00c00000
+#define HEAP_BASE		(SKIBOOT_BASE + 0x00500000)
+#define HEAP_SIZE		0x00b00000
 
 /* This is the location of our console buffer at base + 16M */
 #define INMEM_CON_START		(SKIBOOT_BASE + 0x01000000)
diff --git a/skiboot.lds.S b/skiboot.lds.S
index 18e89f3e73c6..4a7727dc99e1 100644
--- a/skiboot.lds.S
+++ b/skiboot.lds.S
@@ -117,7 +117,7 @@ SECTIONS
 		KEEP(*(.init_array))
 		__ctors_end = .;
 	}
- 
+
 	. = ALIGN(0x10);
 	.opd : {
 		*(.opd)
@@ -201,8 +201,8 @@ SECTIONS
 		*(.branch_lt)
 	}
 
-	/* We locate the BSS at 3M to leave room for the symbol map */
-	. = 0x300000;
+	/* We locate the BSS at 4M to leave room for the symbol map */
+	. = 0x400000;
 
 	_sbss = .;
 	.bss : {
-- 
2.21.0



More information about the Skiboot mailing list