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: kernel: fatal: add explicit no user-mode test case #16854

Conversation

ioannisg
Copy link
Member

We would like to test the HW stack protection feature in builds
with no user-mode support, i.e. CONFIG_USERSPACE=n. For that
we add a new test-case in tests/kernel/fatal test suite.

Signed-off-by: Ioannis Glaropoulos Ioannis.Glaropoulos@nordicsemi.no

@ioannisg ioannisg requested review from andrewboie and dcpleung June 16, 2019 10:10
@ioannisg ioannisg requested review from andyross and nashif as code owners June 16, 2019 10:10
@ioannisg ioannisg added area: Memory Protection Enhancement Changes/Updates/Additions to existing features area: Testing labels Jun 16, 2019
@ioannisg
Copy link
Member Author

@andrewboie is this the right approach to be testing MPU stack guards' feature in builds with no user mode?

@zephyrbot zephyrbot added area: Tests Issues related to a particular existing or missing test area: Kernel labels Jun 16, 2019
We would like to test the HW stack protection feature in ARM
builds with no user-mode support, i.e. CONFIG_USERSPACE=n. For
that we add a new test-case in tests/kernel/fatal test suite.

Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
@ioannisg ioannisg force-pushed the tests_fatal_add_no_usermode_test branch from 8957e40 to 8f4c47a Compare June 16, 2019 21:48
@ioannisg ioannisg added the area: ARM ARM (32-bit) Architecture label Jun 16, 2019
@@ -0,0 +1,5 @@
CONFIG_HW_STACK_PROTECTION=y
CONFIG_ZTEST=y
CONFIG_COVERAGE=n
Copy link
Contributor

Choose a reason for hiding this comment

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

why is COVERAGE explicitly disabled?

Copy link
Member Author

Choose a reason for hiding this comment

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

no particular reason @andrewboie , I suppose it is the same in the default prj.conf

@ioannisg ioannisg requested a review from andrewboie June 19, 2019 10:32
@andrewboie andrewboie merged commit 8004082 into zephyrproject-rtos:master Jun 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM ARM (32-bit) Architecture area: Kernel area: Memory Protection area: Tests Issues related to a particular existing or missing test Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants