Repository navigation
Four buttons, one incident type each - #8
Open
thomas-roos wants to merge 2 commits into
Open
thomas-roos wants to merge 2 commits into
thomas-roos wants to merge 2 commits into
Conversation
platform.exein.io now accepts heap_drop, stack_low and log findings, so the workshop triggers all four Zephyr incident types: - Button 1: 5 identical LOG_ERR -> Log Event (one incident) - Button 2: heap burst -> Heap Exhaustion (unchanged) - Button 3: thread "stack_hog" at ~85% of its stack for 20 s -> Free Stack Below Critical Threshold - Button 4: 160 x 32 B leak over 16 s, freed afterwards -> Memory Leak prj.conf enables POLICY_P2/STACK_LOW/LOG, THREAD_STACK_WARN_PCT=80 and MEM_LEAK_WINDOW_EVENTS=16. Not complete: those options are compiled into the runtime-zephyr blob, which still has HEAP_LOW only. It must be rebuilt against NCS v3.1 before Buttons 1, 3 and 4 produce incidents. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Rebuilt against NCS v3.1.0 from upstream ec8d57d plus the issue #3 network.c fix (version ec8d57d-dirty, as before), protobuf 56d7242, with the app/prj.conf monitor block of the previous commit. Only the blob and its sha256 in module.yml change. Built with scripts/build.sh (nuSIM check passed), not yet tested on a board. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
platform.exein.io now accepts
heap_drop,stack_lowandlogfindings, so each DK button triggers one Zephyr incident type.LOG_ERRstack_hogat ~85% of its 1 KB stack for 20 sapp/prj.conf:POLICY_P2,POLICY_STACK_LOW,POLICY_LOGon;THREAD_STACK_WARN_PCT=80,MEM_LEAK_WINDOW_EVENTS=16runtime-zephyr/: blob rebuilt against NCS v3.1.0 from upstream ec8d57d + the issue Upstream: net_mgmt L4 handler uses 64-bit event type, breaks on NCS v3.1 (blob built with the fix) #3network.cfix (as before), protobuf 56d7242; sha256 updated inmodule.yml. The first commit's title says "blob rebuild pending": the second commit is that rebuild.Same change as exein-io/dots-zephyr-runtime#1 (there against NCS v3.3.1).
Testing:
scripts/build.shbuilds, links the new blob, nuSIM region check passes. Not yet run on a board. Before merging (av*tag publishes to the web flasher), check:LOG_ERR, including modem/LTE errors) and no stack incident (80% applies to all threads)🤖 Generated with Claude Code