Hi,
On 20 January 2014 03:30, Alexey Brodkin <abrod...@synopsys.com> wrote:
>
> This is only required for "PIC" relocation and doesn't apply to modern
> "PIE" relocation which does data relocation as well as code.
>
> "init_sequence_r" is just an array that consists of compile-time
> adresses of init functions. Since this is basically an array of integers
> (pointers to "void" to be more precise) it won't be modified during
> relocation - it will be just copied to new location as it is.
>
> As a consequence on execution after relocation "initcall_run_list" will
> be jumping to pre-relocation addresses. As long as we don't overwrite
> pre-relocation memory area init calls are executed correctly. But still
> it is dangerous because after relocation we don't expect initially used
> memory to stay untouched.
>
> Signed-off-by: Alexey Brodkin <abrod...@synopsys.com>
It's unfortunate that we need to carry over CONFIG_NEEDS_MANUAL_RELOC
to the new common board init code. But if we do, this patch looks
reasonable, so:
Acked-by: Simon Glass <s...@chromium.org>
>
> Cc: Tom Rini <tr...@ti.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Masahiro Yamada <yamad...@jp.panasonic.com>
> Cc: Doug Anderson <diand...@chromium.org>
> Cc: Thomas Langer <thomas.lan...@lantiq.com>
> Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>
>
> Changes to v2:
> * Use ARRAY_SIZE as suggested by Simon Glass
> * Added CONFIG_NEEDS_MANUAL_RELOC wrapper for temporary index variable
> "i" to fix compile-time warning for boards without
> CONFIG_NEEDS_MANUAL_RELOC
> ---
> common/board_r.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/common/board_r.c b/common/board_r.c
> index 86ca1cb..c2d0763 100644
> --- a/common/board_r.c
> +++ b/common/board_r.c
> @@ -903,9 +903,19 @@ init_fnc_t init_sequence_r[] = {
>
> void board_init_r(gd_t *new_gd, ulong dest_addr)
> {
> +#ifdef CONFIG_NEEDS_MANUAL_RELOC
> + int i;
> +#endif
> +
> #ifndef CONFIG_X86
> gd = new_gd;
> #endif
> +
> +#ifdef CONFIG_NEEDS_MANUAL_RELOC
> + for (i = 0; i < ARRAY_SIZE(init_sequence_r); i++)
> + init_sequence_r[i] += gd->reloc_off;
> +#endif
> +
> if (initcall_run_list(init_sequence_r))
> hang();
>
> --
> 1.8.4.2
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
