misc: Optimize flash and memory usage. #270
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Author
|
FYI @pillo79 @pennam for some reason enabling log level=4 (debug for everything) causes a hardfault on Portenta-H7. It shouldn't be a stack overflow issue, but I tried increasing all stacks anyway and it doesn't help. That said, it is very convenient and I have been using it locally, instead of adding debug options manually to conf files, so I will get it fixed somehow and let you know. |
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
0df6fb2 to
46d57bd
Compare
Author
|
Actually, it turns out that it's just some log thread overflow, fixable by increasing stack sizes. However,, I decided to Not enable debug logs globally, because we don't really have extra memory for that and it makes it much harder to see the debug log you're looking for. |
iabdalkader
force-pushed
the
add_debug_overlay
branch
2 times, most recently
from
0cf7454 to
ccd08d2
Compare
iabdalkader
changed the title
iabdalkader
force-pushed
the
add_debug_overlay
branch
2 times, most recently
from
3fed70a to
5d1e8f5
Compare
Add debug config fragment to easily build debug images. Usage: ./extra/build.sh <board> --debug The build includes extra/debug.conf which enables: - Debug mode with assertions and verbose output - Debug-level logging (level 4) - Stack overflow detection - Thread monitoring - Debug optimizations (-Og) - Enable shell. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
The created sketch log structures depends on the config: CONFIG_LOG (IMMEDIATE/DEFERRED): creates log_const_sketch struct. CONFIG_LOG_RUNTIME_FILTERING (implied by CONFIG_SHELL): creates log_dynamic_sketch struct. CONFIG_LOG_MODE_IMMEDIATE + CONFIG_SHELL: creates both. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
- Disable CPP - Disable shell - Disable any debug log level - Disable init stacks. Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
iabdalkader
force-pushed
the
add_debug_overlay
branch
from
5d1e8f5 to
41e6e93
Compare
pennam
approved these changes
Dec 9, 2025
pillo79
approved these changes
Dec 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR optimizes flash and memory usage.
Changes:
To build debug images use:
./extra/build.sh <board> --debugThe build includes extra/debug.conf which enables:
Needs: