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.

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

tests/pkg_utensor: increase main threads stack size [backport 2022.10] #18959

Merged

Conversation

benpicco
Copy link
Contributor

Backport of #18952

Contribution description

The MPU based stack guard is very unpleased by the stack overflow happening during the test. The increase in stack size makes the MPU stack guard happy again.

Testing procedure

make BOARD=nrf52840dk -C tests/pkg_utensor flash test

In master, the test fails with a stack overflow being detected. In this PR, it succeeds.

Issues/PRs references

None

The MPU based stack guard is very unpleased by the stack overflow
happening during the test. The increase in stack size makes the MPU
stack guard happy again.

(cherry picked from commit 22f882d)
@benpicco benpicco added 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 Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Nov 23, 2022
@benpicco benpicco requested a review from maribu November 23, 2022 14:22
@riot-ci
Copy link

riot-ci commented Nov 23, 2022

Murdock results

✔️ PASSED

5d95045 tests/pkg_utensor: increase main threads stack size

Success Failures Total Runtime
221 0 221 02m:02s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

@maribu maribu merged commit 5be77cf into RIOT-OS:2022.10-branch Nov 23, 2022
@benpicco benpicco deleted the backport/2022.10/tests/pkg_utensor branch November 23, 2022 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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 Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants