-
Notifications
You must be signed in to change notification settings - Fork 3k
NRF52832: Extend idle thread stack size to 512 bytes. #4736
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
Conversation
Fix a crash where the idle thread stack size overflows. This crash was depending on the compiler and standard library used.
/morph test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix works for TARGET_MCU_NRF52832
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
I am still having this issue on the NRF52832. Increasing it to 768 works so far. I was wondering, why is there no guard like in mbed_rtx_conf.h? |
@thinkberg Please create an issue with details how to reproduce the issue. |
I would like to, but I may have a questions regarding that, like what influences the stack usage of the idle task, so I can try to reproduce without exposing all code. |
The compiler, its version and the compilation options are the main cause of idle thread stack size overflows. Until static analysis is used to determine the maximum depth of the idle thread stack on every build, I'm afraid there is no reliable solution. |
Then it would make sense to have a guard and be able to set the size in the config, because right now I have to patch mbed-os directly. |
Description
Fix a crash where the idle thread stack size overflows. This crash
was depending on the compiler and standard library used.
Status
READY
Migrations
NO