Skip to content

tests: arch: arm: increase main stack size for no-opt interrupt test #20215

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

Merged
merged 1 commit into from
Nov 5, 2019

Conversation

pabigot
Copy link
Collaborator

@pabigot pabigot commented Oct 29, 2019

Tests fail with the default 512 byte main stack size. Increase it
just as the idle stack size has been increased over default.

Tested on sam_e70_xplained and nrf52840_pca10056 which exhibited the same failure.

Closes #20202

@pabigot pabigot requested a review from ioannisg as a code owner October 29, 2019 11:34
@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label Oct 29, 2019
Copy link
Member

@ioannisg ioannisg left a comment

Choose a reason for hiding this comment

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

Now that I think about it @pabigot, if the reason for this change is the sam e70 failure, we better set the stack size to 1024. For Armv7m platforms with memory protection stack sizes are rounded up to next power of 2 anyways

Tests fail with the default 512 byte main stack size.  Increase it
just as the idle stack size has been increased over default.

Closes zephyrproject-rtos#20202

Signed-off-by: Peter Bigot <[email protected]>
@pabigot
Copy link
Collaborator Author

pabigot commented Nov 1, 2019

set the stack size to 1024

Done.

@ioannisg ioannisg added this to the v2.1.0 milestone Nov 1, 2019
@ioannisg ioannisg added the bug The issue is a bug, or the PR is fixing a bug label Nov 1, 2019
@ioannisg
Copy link
Member

ioannisg commented Nov 5, 2019

@galak ping for taking a quick look :)

@MaureenHelm MaureenHelm merged commit 9003d6c into zephyrproject-rtos:master Nov 5, 2019
@pabigot pabigot deleted the pr/20191029a branch November 12, 2019 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tests/arch/arm/arm_interrupt failed on sam_e70_xplained board.
5 participants