diff mbox

[U-Boot] arm64: Add NOLOAD attribute NOLOAD to .bss sections

Message ID 31889529d98a42c6d2d6195dd37bf27241526288.1496046411.git.michal.simek@xilinx.com
State Accepted
Commit 439edf612047f343737a449e072898b8c6a0b515
Delegated to: Tom Rini
Headers show

Commit Message

Michal Simek May 29, 2017, 8:26 a.m. UTC
Mark explicitly bss sections to not be loaded at
run time.
The similar patch was done in past by:
"Fix linker scripts: add NOLOAD atribute to .bss/.sbss sections"
(sha1: 64134f011254123618798ff77c42ba196b2ec485)

The problem is related to latest toolchain added to Xilinx
v2017.1 design tools where jtag loader is trying to access
ununitialized memory.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/cpu/armv8/u-boot-spl.lds | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Tom Rini June 6, 2017, 12:23 a.m. UTC | #1
On Mon, May 29, 2017 at 10:26:53AM +0200, Michal Simek wrote:

> Mark explicitly bss sections to not be loaded at
> run time.
> The similar patch was done in past by:
> "Fix linker scripts: add NOLOAD atribute to .bss/.sbss sections"
> (sha1: 64134f011254123618798ff77c42ba196b2ec485)
> 
> The problem is related to latest toolchain added to Xilinx
> v2017.1 design tools where jtag loader is trying to access
> ununitialized memory.
> 
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/arch/arm/cpu/armv8/u-boot-spl.lds b/arch/arm/cpu/armv8/u-boot-spl.lds
index cc427c3583fd..0d1b0c499304 100644
--- a/arch/arm/cpu/armv8/u-boot-spl.lds
+++ b/arch/arm/cpu/armv8/u-boot-spl.lds
@@ -56,17 +56,17 @@  SECTIONS
 
 	_image_binary_end = .;
 
-	.bss_start : {
+	.bss_start (NOLOAD) : {
 		. = ALIGN(8);
 		KEEP(*(.__bss_start));
 	} >.sdram
 
-	.bss : {
+	.bss (NOLOAD) : {
 		*(.bss*)
 		 . = ALIGN(8);
 	} >.sdram
 
-	.bss_end : {
+	.bss_end (NOLOAD) : {
 		KEEP(*(.__bss_end));
 	} >.sdram