[U-Boot] [PATCH v5 2/3] arm: make _end compiler-generated

[Benoît Thébaudeau]

Hi Albert,

On Thursday, January 30, 2014 11:02:46 AM, Albert ARIBAUD wrote:
> This prevents references to _end from generating absolute
> relocation records.
> 
> This change is binary invariant for ARM targets.
> 
> Signed-off-by: Albert ARIBAUD <albert.u.boot at aribaud.net>
> ---
> 
> Changes in v5:
> - Rebase onto u-boot-arm/master
> 
> Changes in v4: None
> Changes in v3:
> - Replace _end in linker scripts with _image_binary_end
> 
> Changes in v2: None

[...]

> diff --git a/arch/arm/cpu/u-boot.lds b/arch/arm/cpu/u-boot.lds
> index 4da5d24..f61ef50 100644
> --- a/arch/arm/cpu/u-boot.lds
> +++ b/arch/arm/cpu/u-boot.lds
> @@ -60,7 +60,12 @@ SECTIONS
>  		*(.__rel_dyn_end)
>  	}
>  
> -	_end = .;
> +	.end :
> +	{
> +		*(.__end)
> +	}
> +
> +	_image_binary_end = .;
>  
>  	/*
>  	 * Deprecated: this MMU section is used by pxa at present but
> @@ -91,7 +96,9 @@ SECTIONS
>  		KEEP(*(.__bss_end));
>  	}
>  
> -	.dynsym _end : { *(.dynsym) }
> +	.dynsym _image_binary_end : { *(.dynsym) }
> +	.hash : { *(.hash) }
> +	.got.plt : { *(.got.plt) }

Are the two additions above intentional? These lines were present in the context
in v4, but not as an addition. They are also unrelated to the topic of this
patch.

>  	.dynbss : { *(.dynbss) }
>  	.dynstr : { *(.dynstr*) }
>  	.dynamic : { *(.dynamic*) }

[...]

Best regards,
Benoît