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

tests/pthread_flood: increase stack size #18916

Closed
wants to merge 1 commit into from
Closed

tests/pthread_flood: increase stack size #18916

wants to merge 1 commit into from

Conversation

maribu
Copy link
Member

@maribu maribu commented Nov 15, 2022

Contribution description

This fixes stack overflows while printing the stack consumption when newlib is used.

Testing procedure

$ make BOARD=nucleo-f767zi -C tests/pthread_flood flash test

Should no longer result in a hard fault

Issues/PRs references

None

@maribu
tests/pthread_flood: increase stack size
fd83529 
This fixes stack overflows while printing the stack consumption when
newlib is used.
@maribu maribu added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Nov 15, 2022
@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Nov 15, 2022
@maribu maribu mentioned this pull request Nov 15, 2022
Merged
@riot-ci
Copy link

riot-ci commented Nov 15, 2022

Murdock results

❌ FAILED

fd83529 tests/pthread_flood: increase stack size

Build failures (10)
Application Target Toolchain Runtime (s) Worker
tests/pthread_flood i-nucleo-lrwan1 gnu 2.11 beast
tests/pthread_flood bluepill-stm32f030c8 gnu 2.66 mobi3
tests/pthread_flood nucleo-f042k6 gnu 2.07 breeze
tests/pthread_flood nucleo-f030r8 gnu 3.27 mobi3
tests/pthread_flood nucleo-l053r8 gnu 2.41 beast
tests/pthread_flood nucleo-l031k6 gnu 3.50 mobi3
tests/pthread_flood stm32l0538-disco gnu 1.57 breeze
tests/pthread_flood stm32f0discovery gnu 2.80 mobi3
tests/pthread_flood stm32g0316-disco gnu 1.97 riotbuild
tests/pthread_flood slstk3400a gnu 7.55 riotbuild

Artifacts

@maribu maribu mentioned this pull request Nov 17, 2022
Merged
@maribu
Copy link
Member Author

maribu commented Dec 11, 2022

Not needed anymore

@maribu maribu closed this Dec 11, 2022
@maribu maribu deleted the tests/pthread_flood branch December 11, 2022 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smlng smlng Awaiting requested review from smlng

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

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

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

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

Assignees
No one assigned
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maribu @riot-ci