From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew F. Davis Date: Wed, 11 Jan 2017 14:19:03 -0600 Subject: [U-Boot] [RFC] spl: Remove overwrite of relocated malloc limit Message-ID: <20170111201903.17518-1-afd@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de spl_init on some boards is called after stack and heap relocation, on some platforms spl_relocate_stack_gd is called to handle setting the limit to its value CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN when simple SPL malloc is enabled during relocation. spl_init should then not re-assign the old pre-relocation limit. Remove this. Also add some debug statements that can help with find similar problems. Signed-off-by: Andrew F. Davis --- I'm not sure what the real fix should be, probably to clean-up all this spaghetti code relating to malloc, but for now without this patch we are stuck with the pre-relocation heap size and boot with simple malloc is not functional on AM335x-EVM and probably many other similar boards. common/malloc_simple.c | 6 +++++- common/spl/spl.c | 1 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/common/malloc_simple.c b/common/malloc_simple.c index 0f6bcbcc71..611400265b 100644 --- a/common/malloc_simple.c +++ b/common/malloc_simple.c @@ -39,10 +39,14 @@ void *memalign_simple(size_t align, size_t bytes) addr = ALIGN(gd->malloc_base + gd->malloc_ptr, align); new_ptr = addr + bytes - gd->malloc_base; - if (new_ptr > gd->malloc_limit) + if (new_ptr > gd->malloc_limit) { + debug("space exhausted\n"); return NULL; + } + ptr = map_sysmem(addr, bytes); gd->malloc_ptr = ALIGN(new_ptr, sizeof(new_ptr)); + debug("%lx\n", (ulong)ptr); return ptr; } diff --git a/common/spl/spl.c b/common/spl/spl.c index a76ea3a603..aa75c93fce 100644 --- a/common/spl/spl.c +++ b/common/spl/spl.c @@ -237,7 +237,6 @@ int spl_init(void) #ifdef CONFIG_MALLOC_F_ADDR gd->malloc_base = CONFIG_MALLOC_F_ADDR; #endif - gd->malloc_limit = CONFIG_SYS_MALLOC_F_LEN; gd->malloc_ptr = 0; #endif if (CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)) { -- 2.11.0