Skip to content

Revert "Allow max-depth specification in memap" #3003

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
Oct 12, 2016

Conversation

c1728p9
Copy link
Contributor

@c1728p9 c1728p9 commented Oct 12, 2016

This reverts commit 8f9f4ee since it broke memory map info for IAR, ARMCC and on windows, GCC_ARM.

This reverts commit 8f9f4ee since
it broke memory map info for IAR, ARMCC and on windows, GCC_ARM.
@c1728p9
Copy link
Contributor Author

c1728p9 commented Oct 12, 2016

/morph test

@c1728p9
Copy link
Contributor Author

c1728p9 commented Oct 12, 2016

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 0

All builds and test passed!

@bridadan
Copy link
Contributor

The memap breakdown of the tests is all wacky, but that's probably due to the way we build them. They've always been a bit wonky.

But the example build looks ok for ARM and GCC_ARM. Granted these were built on a Linux machine, so

ARM

15:32:56 +------------------------+-------+-------+--------+
15:32:56 | Module                 | .text | .data |   .bss |
15:32:56 +------------------------+-------+-------+--------+
15:32:56 | Misc                   | 12909 |    16 |  16756 |
15:32:56 | drivers                |   734 |     4 |     28 |
15:32:56 | hal                    |   564 |     8 |      0 |
15:32:56 | platform               |  2467 |     8 |     92 |
15:32:56 | rtos                   |   184 |     8 |      0 |
15:32:56 | rtos/rtx               | 10516 |    92 |  23732 |
15:32:56 | targets/TARGET_RENESAS | 15924 |   203 | 609772 |
15:32:56 | Subtotals              | 43298 |   339 | 650380 |
15:32:56 +------------------------+-------+-------+--------+
15:32:56 Allocated Heap: unknown
15:32:56 Allocated Stack: unknown
15:32:56 Total Static RAM memory (data + bss): 650719 bytes
15:32:56 Total RAM memory (data + bss + heap + stack): 650719 bytes
15:32:56 Total Flash memory (text + data + misc): 43637 bytes

GCC_ARM

15:32:20 +------------------------+-------+-------+-------+
15:32:20 | Module                 | .text | .data |  .bss |
15:32:20 +------------------------+-------+-------+-------+
15:32:20 | Fill                   |    35 |     5 |    11 |
15:32:20 | Misc                   | 20240 |  2176 |   108 |
15:32:20 | cmsis/TOOLCHAIN_GCC    |   180 |     0 |     0 |
15:32:20 | hal                    |   608 |     0 |     8 |
15:32:20 | platform               |  1686 |     4 |   269 |
15:32:20 | rtos                   |    56 |     4 |     4 |
15:32:20 | rtos/rtx               |  9308 |    21 | 21099 |
15:32:20 | targets/TARGET_RENESAS | 13003 |     6 |  2885 |
15:32:20 | Subtotals              | 45116 |  2216 | 24384 |
15:32:20 +------------------------+-------+-------+-------+
15:32:20 Allocated Heap: 524288 bytes
15:32:20 Allocated Stack: 82688 bytes
15:32:20 Total Static RAM memory (data + bss): 26600 bytes
15:32:20 Total RAM memory (data + bss + heap + stack): 633576 bytes
15:32:20 Total Flash memory (text + data + misc): 47332 bytes

Some of the Modules are different, but still is more consistent than before.

@bridadan bridadan closed this Oct 12, 2016
@bridadan bridadan reopened this Oct 12, 2016
@bridadan
Copy link
Contributor

Oops, wrong button :)

@sg- sg- merged commit 1ba3499 into ARMmbed:master Oct 12, 2016
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.

4 participants