[NCS36510] Reduce default heap size allocated by IAR to 1/4 of RAM #4187
Conversation
…RAM. Allows mesh minimal example to compile with IAR.
|
@maclobdell no issues from me. The networking team will find any possible issues as they add the device to large networks as a router (routing tables = more RAM I'm assuming) |
|
@maclobdell Adding @danish-iftikhar |
|
Heap should be set dynamically to 'remaining memory' for GCC and ARM, as far as i know it's impossible to do that for IAR. So it's more or less per platform exercise to maintain balance and don't waste too much memory. @0xc0170 |
|
Actually a lot of platforms hard code it to 1/4 of RAM not only for IAR. |
|
/morph test |
1 similar comment
|
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
…lwip_broadcast Release mbed OS 5.4.5 and mbed lib v142 Ports for Upcoming Targets Fixes and Changes 4059: [Silicon Labs] Rename targets ARMmbed#4059 4115: Support for Qt Creator Generic project export and associated Makefile ARMmbed#4115 3915: Feature vscode ARMmbed#3915 4205: tests: race test - add not supported for single threaded env ARMmbed#4205 4187: [NCS36510] Reduce default heap size allocated by IAR to 1/4 of RAM ARMmbed#4187 4145: test - add nanostack to examples.json file ARMmbed#4145 4093: Update.py: New feature - update a branch instead of a fork, plus general improvements. ARMmbed#4093 4225: fixed missing device_name for xDot and removed progen ARMmbed#4225 4243: Config: config header file should contain new line ARMmbed#4243 4251: Fix C++11 build error w/ u-blox EVK-ODIN-W2 ARMmbed#4251 4236: STM32 Fixed warning related to __packed redefinition ARMmbed#4236 4224: Add `mbed new .` output to export ARMmbed#4224 4190: LPC4088: Enable LWIP feature ARMmbed#4190 4136: Error when bootloader is specified but does not exist ARMmbed#4136 3881: Remove debug links to printf/exit in NDEBUG builds ARMmbed#3881 4260: Inherit Xadow M0 target from LPC11U35_501 ARMmbed#4260 4249: Add consistent button names across targets ARMmbed#4249 4254: Removed unused variable in TARGET_NXP/lpc17_emac.c ARMmbed#4254
Ports for Upcoming Targets Fixes and Changes 4059: [Silicon Labs] Rename targets ARMmbed/mbed-os#4059 4187: [NCS36510] Reduce default heap size allocated by IAR to 1/4 of RAM ARMmbed/mbed-os#4187 4225: fixed missing device_name for xDot and removed progen ARMmbed/mbed-os#4225 4251: Fix C++11 build error w/ u-blox EVK-ODIN-W2 ARMmbed/mbed-os#4251 4236: STM32 Fixed warning related to __packed redefinition ARMmbed/mbed-os#4236 4190: LPC4088: Enable LWIP feature ARMmbed/mbed-os#4190 4260: Inherit Xadow M0 target from LPC11U35_501 ARMmbed/mbed-os#4260 4249: Add consistent button names across targets ARMmbed/mbed-os#4249
Description
The IAR linker file allocates the heap size by default to 0x4000, which for this target is causing the mbed-os-example-mesh-minimal application not to be able to fit into the RAM. It has been changed to 0x3000 (12K, which is 1/4 of the total 48K RAM). This change both passes all mbed OS tests and allows the mesh minimal example to compile with IAR and run.
helps fix ARMmbed/mbed-os-example-mesh-minimal#63
cc: @pradeep-gr @radhika-raghavendran @danclement @jacobjohnson-ON
please review and confirm this looks good to you.
Status
READY
Migrations
NO
Related PRs
None
Todos
none
Deploy notes
none
Steps to test or reproduce
n/a