diff mbox

[U-Boot] arm: atmel: cpux9k2: increase malloc space to fix crash on start u-boot

Message ID 1379513592-5470-1-git-send-email-esw@bus-elektronik.de
State Superseded, archived
Delegated to: Andreas Bießmann
Headers show

Commit Message

esw@bus-elektronik.de Sept. 18, 2013, 2:13 p.m. UTC
From: "Jens Scharsig (BuS Elektronik)" <esw@bus-elektronik.de>

Since UBIFS is enabled for cpux9k2, more malloc space is needed.
For the current uboot 2013.10-rcX the size is to small, this will fix the
startup problems by increasing the mallog space.

Signed-off-by: Jens Scharsig (BuS Elektronik) <esw@bus-elektronik.de>
---
 include/configs/eb_cpux9k2.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andreas Bießmann Sept. 18, 2013, 2:44 p.m. UTC | #1
Dear Jens Scharsig,

On 09/18/2013 04:13 PM, Jens Scharsig (BuS Elektronik) wrote:
> From: "Jens Scharsig (BuS Elektronik)" <esw@bus-elektronik.de>
> 
> Since UBIFS is enabled for cpux9k2, more malloc space is needed.
> For the current uboot 2013.10-rcX the size is to small, this will fix the
> startup problems by increasing the mallog space.
> 
> Signed-off-by: Jens Scharsig (BuS Elektronik) <esw@bus-elektronik.de>
> ---
>  include/configs/eb_cpux9k2.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/eb_cpux9k2.h b/include/configs/eb_cpux9k2.h
> index ccc7bd0..7aca74b 100644
> --- a/include/configs/eb_cpux9k2.h
> +++ b/include/configs/eb_cpux9k2.h
> @@ -83,7 +83,7 @@
>   * Size of malloc() pool
>   */
>  
> -#define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + 520*1024)
> +#define CONFIG_SYS_MALLOC_LEN		(1024 * 1024)

are you sure 1MiB is enough? We had from time to time problems
initializing an unsafe closed UBI (about 512MiB). With 4MiB malloc arena
however we never had these problems.

Best regards

Andreas Bießmann
Jens Scharsig Sept. 18, 2013, 3:56 p.m. UTC | #2
Dear Andreas Bießmann:
> 
> are you sure 1MiB is enough? We had from time to time problems
> initializing an unsafe closed UBI (about 512MiB). With 4MiB malloc arena
> however we never had these problems.

Thank you for informations. To prevent future problems, I will send a
4MiB version (V2) tomorrow.

Regards

Jens
diff mbox

Patch

diff --git a/include/configs/eb_cpux9k2.h b/include/configs/eb_cpux9k2.h
index ccc7bd0..7aca74b 100644
--- a/include/configs/eb_cpux9k2.h
+++ b/include/configs/eb_cpux9k2.h
@@ -83,7 +83,7 @@ 
  * Size of malloc() pool
  */
 
-#define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + 520*1024)
+#define CONFIG_SYS_MALLOC_LEN		(1024 * 1024)
 
 /*
  * sdram