Skip to content

Conversation

@ioannisg
Copy link
Member

@ioannisg ioannisg commented Dec 9, 2020

Modify test .yaml file, to allow the .gap_filling test
variant to execute only on cortex-m33 platforms with
sufficient number of MPU regions. Copy pasting the
configuration from mem_protect/userspace test.

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

Fixes #30253

@github-actions github-actions bot added area: Kernel area: Tests Issues related to a particular existing or missing test labels Dec 9, 2020
@ioannisg ioannisg requested a review from PerMac December 9, 2020 16:58
@ioannisg ioannisg added the platform: nRF Nordic nRFx label Dec 9, 2020
Copy link
Member

@PerMac PerMac left a comment

Choose a reason for hiding this comment

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

I think you were a bit too generous with copy pasting ;) Now the tests in this yaml will have the same names/ids as the ones in userspace, and it shouldn't be the case, so probably you'll have to remove .userspace from the names

Modify test .yaml file, to allow the .gap_filling test
variant to execute only on cortex-m33 platforms with
sufficient number of MPU regions. Copy pasting the
configuration from mem_protect/userspace test.

Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
@ioannisg ioannisg force-pushed the tests_mem_protect_exclude_nrf5340_dk branch from 48c58b9 to b5a597b Compare December 10, 2020 08:23
@ioannisg ioannisg requested a review from PerMac December 10, 2020 08:25
@ioannisg ioannisg self-assigned this Dec 10, 2020
@ioannisg ioannisg added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Dec 10, 2020
@carlescufi carlescufi merged commit 4f4dd9f into zephyrproject-rtos:master Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Kernel area: Tests Issues related to a particular existing or missing test platform: nRF Nordic nRFx Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests: kernel: Test kernel.memory_protection.gap_filling fails on nrf5340dk_nrf5340_cpuapp

3 participants