-
Notifications
You must be signed in to change notification settings - Fork 178
parser for MBED_MEM_TRACE output #630
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
Comments
@thinkberg You're welcome to submit a PR. I'm not sure mbed CLI would be the best place for this. @screamerbg Do you have any thoughts on that subject? |
I remember something like I didn't know which is the best part to submit something too, so I wanted to get some feedback. It felt more home in mbed-cli than in the mbed-os part. |
@thinkberg I agree with @theotherjimmy that Mbed CLI might not be the best place for the parsing. Do you think that this could be something like an external tool or python module, something that Mbed CLI could call? Or perhaps be part of Mbed OS tools given that many tools inside the OS have knowledge about the target, ram/flash, regions, even flash algos, device IDs, etc. |
Sure, could be an external module that can be called. My reasoning was that its important enough to be easily accessible. Right now, having a memory leak involves quite some searching and guessing. Even the stack/thread debug that I referenced in the repo was a lucky find. You almost always find outdates mbed docs, unfortunately. Even if it would be part of the docs with reference to the external tools would be most helpful. |
@thinkberg @theotherjimmy Mbed CLI now has a serial terminal feature (#664). Perhaps this is something that could be added to the serial terminal hooks and enable runtime memtrace functionality? Note that Mbed CLI does not have any external python dependencies, so the pygtail dependency should be somehow workarounded. |
Actually, pygtail was just a workaround due to buffering issues. If the mbed cli serial can pipe its data into a stream it would be simple to make it work. |
If you give me a hint where to integrate (where are the serial terminal hooks?), I can port it to be part of the cli. |
@thinkberg The file you're looking for is: https://github.com/ARMmbed/mbed-cli/blob/master/mbed/mbed_terminal.py Let us know if you have any questions. |
I will take a look, thanks. |
Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-61 |
Started working on it. |
PR in #789 |
Thank you for raising this issue. Please note we have updated our policies and |
Description
Enhancement
The current memtrace is a very nice feature, but rather hard to use. Since my forum post is now in moderation for a week, maybe you'd like to integrate my little script into the cli tooling itself?
https://github.com/thinkberg/mbed-memtrace-logger
It parses the MEM_TRACE output and displays allocated heap and +/- values in a more readable form to aid debugging, while keeping log functionality.
Leo.
The text was updated successfully, but these errors were encountered: