Skip to content

Fix Stack stats by running the test command with "-DMBED_HEAP_STATS_ENABLED=1" #3624

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
Feb 2, 2017

Conversation

TomoYamanaka
Copy link
Contributor

In CI, I receive an mbed assert of stack stats by running the following commands. Its assert is TIMEOUT.

mbed test -m RZ_A1H -t GCC_ARM -DMBED_STACK_STATS_ENABLED=1 --clean --compile
mbed test -m RZ_A1H -t GCC_ARM --run -n tests-mbed_drivers-stl_features -v

So, I fixed the process of related to stack stats (osThreadInfoStackMax).The Detail contents is here.
#3273 (comment)

…NABLED=1"

In CI, I receive an mbed assert of stack stats by running the following commands. Its assert is TIMEOUT.
mbed test -m RZ_A1H -t GCC_ARM -DMBED_STACK_STATS_ENABLED=1 --clean --compile
mbed test -m RZ_A1H -t GCC_ARM --run -n tests-mbed_drivers-stl_features -v

So, I fixed the process of related to stack stats (osThreadInfoStackMax).The Detail contents is here.
ARMmbed#3273 (comment)
@bridadan
Copy link
Contributor

Hi @TomoYamanaka, thanks for looking at this! I'm traveling this week and I don't have a PEACH to test with me. I'll verify this next week, but in the meantime I'll run the CI on this.

/morph test

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 1459

All builds and test passed!

@0xc0170 0xc0170 removed the needs: CI label Jan 30, 2017
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 30, 2017

Hi @TomoYamanaka, thanks for looking at this! I'm traveling this week and I don't have a PEACH to test with me. I'll verify this next week, but in the meantime I'll run the CI on this.

Any update @bridadan

Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your patience everybody, finally got a chance to test this. All tests are GREEN! 🎉

Well done @TomoYamanaka! LGTM

@sg- sg- merged commit f163e1e into ARMmbed:master Feb 2, 2017
@bridadan bridadan mentioned this pull request Feb 6, 2017
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.

5 participants