Skip to content

Nano 100: Lower HEAP size for IAR #6768

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 7, 2018
Merged

Conversation

deepikabhavnani
Copy link

@deepikabhavnani deepikabhavnani commented Apr 27, 2018

Description

IAR 7.8 does not support dynamic heap, and some test/addition of new feature fail on this device because of less static RAM memory. Reducing the heap memory size for the same reason.

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

@ccli8 @ccchang12

IAR 7.8 does not support dynamic heap, and some test/addition of new
feature fail on this device because of less static RAM memory.
Reducing the heap memory size for the same reason.
@0xc0170
Copy link
Contributor

0xc0170 commented May 3, 2018

@ccli8 @ccchang12

Can you please review this change?

/morph build

@ccli8
Copy link
Contributor

ccli8 commented May 3, 2018

@deepikabhavnani I send another PR #6799 to spare more memory for your addition of new feature. Per our experience, some Greentea/CI tests would fail due to insufficient heap.

@mbed-ci
Copy link

mbed-ci commented May 3, 2018

Build : SUCCESS

Build number : 1896
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6768/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented May 3, 2018

@mbed-ci
Copy link

mbed-ci commented May 3, 2018

@deepikabhavnani
Copy link
Author

@ccli8 - We have some devices with 0x1000 heap size and 0x400 stack size and looks like we didn;t had any issue with CI with heap change. Is it fine to go ahead with this change?

@0xc0170 0xc0170 changed the title Updating HEAP size Nano 100: Lower HEAP size for IAR May 3, 2018
@ccli8
Copy link
Contributor

ccli8 commented May 4, 2018

@deepikabhavnani OK. Go with 0x1000 heap size and 0x400 stack size.

@ccli8
Copy link
Contributor

ccli8 commented May 4, 2018

@deepikabhavnani Correct above. Go with 0x1000 or 0x1200 heap size and 0x400 stack size only for IAR toolchain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants