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

STM32 increase IAR heap size for big RAM targets #4031

Merged
merged 1 commit into from Apr 20, 2017

Conversation

Projects
None yet
6 participants
@jeromecoutant
Contributor

jeromecoutant commented Mar 27, 2017

Description

Following #3920
I have updated the heap size for all targets with 192K RAM and more

Status

READY

Tests

OS5 results are coming.
OS2 tests:
image

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Mar 27, 2017

STM32F4 O2 results:
image

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 27, 2017

What rule did you follow (how much did you increase the size)? And whats the reason?

IAR is limited in this matter, and heap size is quite application specific, therefore we should avoid editing this numbers (we would just play ping-pong with the size and might not even find a number that fits all)..

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Mar 27, 2017

What rule did you follow (how much did you increase the size)? And whats the reason?

#3920

@mazimkhan

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Mar 27, 2017

Hi
OS5 tests result is OK as expected.
copy/paste OS5 result array looks too big...

@sg- sg- added the needs: CI label Mar 29, 2017

@bridadan

This comment has been minimized.

Contributor

bridadan commented Mar 30, 2017

/morph test-nightly

@mbed-bot

This comment has been minimized.

mbed-bot commented Mar 30, 2017

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1792

Test failed!

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 30, 2017

Restarting, one failure that shall not be relevant for this patch (gcc arm not IAR).

/morph test-nightly

@mbed-bot

This comment has been minimized.

mbed-bot commented Mar 30, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1798

All builds and test passed!

@bridadan bridadan added needs: review and removed needs: CI labels Apr 3, 2017

@sg- sg- added ready for merge and removed needs: review labels Apr 10, 2017

@adbridge adbridge merged commit 6a6455d into ARMmbed:master Apr 20, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test-nightly Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeromecoutant jeromecoutant deleted the jeromecoutant:PR_IAR_BIG_HEAP branch Apr 21, 2017

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