-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Conversation
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.
Can you please review this change? /morph build |
@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. |
Build : SUCCESSBuild number : 1896 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1543 |
Test : SUCCESSBuild number : 1713 |
@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? |
@deepikabhavnani OK. Go with 0x1000 heap size and 0x400 stack size. |
@deepikabhavnani Correct above. Go with 0x1000 or 0x1200 heap size and 0x400 stack size only for IAR toolchain. |
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
@ccli8 @ccchang12