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.

Sign up for GitHub

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

Jump to bottom

cpu/cortexm_common: print last active thread on stack corruption #20709

Merged
merged 1 commit into from
May 30, 2024
Merged

cpu/cortexm_common: print last active thread on stack corruption #20709

merged 1 commit into from
May 30, 2024

Conversation

benpicco
Copy link
Contributor

Contribution description

This might give a hint as to which thread caused the corruption.

Testing procedure

Issues/PRs references

@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: cpu Area: CPU/MCU ports labels May 29, 2024
@benpicco benpicco requested a review from fabian18 May 29, 2024 15:43
fabian18
fabian18 reviewed May 29, 2024
View reviewed changes
Copy link
Contributor

@fabian18 fabian18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I knew you would add this :) Thank you.

cpu/cortexm_common/vectors_cortexm.c Outdated Show resolved Hide resolved
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 29, 2024
@riot-ci
Copy link

riot-ci commented May 29, 2024

Murdock results

✔️ PASSED

00c1f4e cpu/cortexm_common: print last active thread on stack corruption

Success Failures Total Runtime
10161 0 10161 13m:47s

Artifacts

@maribu maribu added this pull request to the merge queue May 30, 2024
Merged via the queue into RIOT-OS:master with commit ea3d4c8 May 30, 2024
27 checks passed
@benpicco benpicco deleted the stack_overflow-thread branch May 30, 2024 14:46
@mguetschow mguetschow added this to the Release 2024.07 milestone Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabian18 fabian18 fabian18 approved these changes

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

@bergzand bergzand Awaiting requested review from bergzand bergzand is a code owner

Assignees
No one assigned
Labels
Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms
Projects
None yet
Milestone
Release 2024.07
Development

Successfully merging this pull request may close these issues.
5 participants
@benpicco @riot-ci @fabian18 @maribu @mguetschow