In case of CONFIG_SYS_MALLOC_F_LEN "malloc_base" is used for early start-up code and is set very early, typically in "start.S" or "crt1.S".
In current implementation in case of CONFIG_SYS_GENERIC_GLOBAL_DATA all global data gets zeroed on "board_init_f" entry. But by that time "malloc_base" could have been set already, which means it will be zeroed and subsequent C-code will be executed improperly (if executed at all - if there's no memory mapped to 0 or it is read-only then on some arches there will be an exception and others will quetly die). To work-around described situation we just need to make sure "malloc_base" is saved prior zeroing global data and recovered afterwards. Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> Cc: Wolfgang Denk <w...@denx.de> Cc: Simon Glass <s...@chromium.org> Cc: Tom Rini <tr...@ti.com> Cc: Masahiro Yamada <yamad...@jp.panasonic.com> --- common/board_f.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/common/board_f.c b/common/board_f.c index 3a4b32c..ebdba0e 100644 --- a/common/board_f.c +++ b/common/board_f.c @@ -999,6 +999,9 @@ static init_fnc_t init_sequence_f[] = { void board_init_f(ulong boot_flags) { #ifdef CONFIG_SYS_GENERIC_GLOBAL_DATA +#ifdef CONFIG_SYS_MALLOC_F_LEN + int malloc_base; +#endif /* * For some archtectures, global data is initialized and used before * calling this function. The data should be preserved. For others, @@ -1009,12 +1012,25 @@ void board_init_f(ulong boot_flags) gd = &data; +#ifdef CONFIG_SYS_MALLOC_F_LEN + /* + * "malloc_base" is supposed to be set in the very beginning of start-up + * code (start.S or crt0.S), now we need to preserve it from zeroing. + */ + malloc_base = gd->malloc_base; +#endif + /* * Clear global data before it is accessed at debug print * in initcall_run_list. Otherwise the debug print probably * get the wrong vaule of gd->have_console. */ zero_global_data(); + +#ifdef CONFIG_SYS_MALLOC_F_LEN + /* Restore "malloc_base" value */ + gd->malloc_base = malloc_base; +#endif #endif gd->flags = boot_flags; -- 2.1.0 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot