Enable split heap in K64F/K66F devices #10950
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
-Enable MBED_SPLIT_HEAP for K64F and K66F -Allow GCC_ARM toolchain to utilize remaining 64K memory area -Make ARM toolchain to start memory filling from 64K region to leave more space to bigger 192K region.
kjbracey
approved these changes
Jul 3, 2019
artokin
added
needs: CI
release-version: 5.13.0-rc2
release-version: 5.13.2
and removed
release-version: 5.13.0-rc2
labels
|
CI started. |
Test run: FAILEDSummary: 1 of 4 test jobs failed Failed test jobs:
|
|
ARM build failed: |
When packing data into multiple regions using the `.ANY` directive, the linker can accidentally overfill an area. This doesn't normally happen because it defaults to `--any_placement=worst_fit`, which puts data in the region with most space. When we prioritise regions with `.ANY1`/`.ANY2`, it may totally fill an area, then fail to leave enough space for linker-generated veneers. We've just seen this error with the new K64F linker map. Adding `--any-contingency` makes it lower priority when a region is 98% full, avoiding this error. The option should not have any effect on targets with scatter files without prioritised `.ANY` directives.
|
Fix provided, CI started. |
Test run: FAILEDSummary: 2 of 11 test jobs failed Failed test jobs:
|
|
Linker still reports a problem |
Move heap_0 after .data as it is also placed to m_data.
|
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
artokin
added
release-version: 5.13.1
and removed
ready for merge
release-version: 5.13.2
labels
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
-Enable MBED_SPLIT_HEAP for K64F and K66F
-Allow GCC_ARM toolchain to utilize remaining 64K memory area
-Make ARM toolchain to start memory filling from 64K region to leave
more space to bigger 192K region
Pull request type
Reviewers
Release Notes