Skip to content
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

greentea: Add reporting of reserved heap #4504

Merged
merged 1 commit into from Jun 19, 2017

Conversation

geky
Copy link
Contributor

@geky geky commented Jun 8, 2017

This exposes the reporting of reserved heap and stack to mbed-greentea when the memory stats are enabled.

Note: Reporting of reserved stack is already performed by the thread layer

related pr ARMmbed/greentea#228
cc @bridadan, @studavekar

@tommikas
Copy link
Contributor

tommikas commented Jun 9, 2017

@geky I think you need to rebase to get the Jenkinsfile update in. jenkins/pr-head currently won't pass against master without it.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

Thanks @tommikas , I'll rebase for him .
Btw, I should create PR to revert that removal and keep it up until we make it green (to track the progress)

@0xc0170 0xc0170 force-pushed the gt-add-reserved-memory-reporting branch from 269c3c3 to 9768bfd Compare June 9, 2017 09:31
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

Rebase done

Note: Reporting of reserved stack is already performed by the thread
layer
@0xc0170 0xc0170 force-pushed the gt-add-reserved-memory-reporting branch from 9768bfd to 40957c1 Compare June 9, 2017 09:57
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.

Love 1 line diffs ❤️

@geky
Copy link
Contributor Author

geky commented Jun 13, 2017

Thanks for the rebase 👍

@sg-
Copy link
Contributor

sg- commented Jun 15, 2017

/morph test

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 564

All builds and test passed!

@theotherjimmy theotherjimmy merged commit a6695ae into ARMmbed:master Jun 19, 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.

None yet

8 participants