Skip to content

foundries.io CI: tests/kernel/mem_protect/stackprot fails on nrf52 #15379

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

Closed
mike-scott opened this issue Apr 12, 2019 · 5 comments · Fixed by #15417
Closed

foundries.io CI: tests/kernel/mem_protect/stackprot fails on nrf52 #15379

mike-scott opened this issue Apr 12, 2019 · 5 comments · Fixed by #15417
Assignees
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 platform: nRF Nordic nRFx priority: medium Medium impact/importance bug
Milestone

Comments

@mike-scott
Copy link
Contributor

mike-scott commented Apr 12, 2019

Describe the bug
Due to PR: 3e255e9
sanitycheck now fails on nrf52_blenano2 -- and probably a lot of other arm HW. (log below in "Screenshots or console output")

Related CI failure:
https://ci.foundries.io/projects/zephyr/builds/5694/sanity-test-nrf52/

To Reproduce
Steps to reproduce the behavior:

  1. cd tests/kernel/mem_protect/stackprot
  2. mkdir build; cd build
  3. cmake -DBOARD=nrf52_blenano2 ..
  4. make flash
  5. See error below

Expected behavior
Pass sanitycheck like previous to this commit

Impact
It's a regression for CI testing.. all of nRF52 and other arm HW will probably fail this test as a result

Screenshots or console output

 94/153 nrf52_blenano2            tests/kernel/mem_protect/stackprot/kernel.memory_protection FAILED: timeout
../tmp/outdir/nrf52_blenano2/tests/kernel/mem_protect/stackprot/kernel.memory_protection/handler.log
�***** Booting Zephyr OS v1.14.0-rc3-230-g1897896e9739 *****
Starts alternate_thread
alternate_thread: Input string is too long and stack overflowed!

***** Stack Check Fail! *****
Current thread ID = 0x20000068
Faulting instruction address = 0x8a54
Fatal fault in thread 0x20000068! Aborting.
Running test suite stackprot
===================================================================
starting test - test_stackprot
***** MPU FAULT *****
  Stacking error (context area might be not valid)
  Data Access Violation
  MMFAR Address: 0x20001c10
***** Stack Check Fail! *****
Current thread ID = 0x20000344
Faulting instruction address = 0x97e0
Fatal fault in essential thread! Spinning...

Environment (please complete the following information):

  • OS: Arch Linux
  • Toolchain: Zephyr SDK 0.10.0
  • Commit SHA: today's master (a942fcc)
@mike-scott mike-scott added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug area: Tests Issues related to a particular existing or missing test labels Apr 12, 2019
@mike-scott mike-scott added this to the v1.14.0 milestone Apr 12, 2019
@aescolar aescolar added the platform: nRF Nordic nRFx label Apr 12, 2019
@aescolar
Copy link
Member

aescolar commented Apr 12, 2019

@mike-scott : From your description this seems to not be related to sanitycheck, but to this particular test? Also, Is this test failing in the Zephyr master branch? (If so, the problem would not be related to your CI)
If that is the case, could you change the title to be a bit more clear that it just relates to this test and particular board?
(Something like "tests/kernel/mem_protect/stackprot fails on nrf52_blenano2" )

@mike-scott
Copy link
Contributor Author

@aescolar sanitycheck runs this for nrf52 hardware. The "easy" way to reproduce is to run just this test. The actual regression was found via sanitycheck for real HW.

I don't see the issue with the description.

@mike-scott
Copy link
Contributor Author

Note: this test will probably fail on a lot more hardware than just nrf52_blenano2.

I'll adjust the title to mention the test specifically.

@mike-scott mike-scott changed the title foundries.io CI: sanitycheck fail on nrf52 foundries.io CI: tests/kernel/mem_protect/stackprot fails on nrf52 Apr 12, 2019
@wentongwu wentongwu assigned wentongwu and unassigned nashif Apr 12, 2019
@wentongwu
Copy link
Collaborator

wentongwu commented Apr 12, 2019

@mike-scott Actually this change passed the CI test before check in. Anyway I will fix it ASAP.

wentongwu added a commit to wentongwu/zephyr that referenced this issue Apr 12, 2019
increase stack size for stackprot test, commit 3e255e9
cause this issue.

Fixes: zephyrproject-rtos#15379.

Signed-off-by: Wentong Wu <[email protected]>
wentongwu added a commit to wentongwu/zephyr that referenced this issue Apr 12, 2019
revert commit 3e255e9 which is to adjust stack size
on qemu_x86 platform for coverage test, but break other
platform's CI test.

Fixes: zephyrproject-rtos#15379.

Signed-off-by: Wentong Wu <[email protected]>
wentongwu added a commit to wentongwu/zephyr that referenced this issue Apr 12, 2019
revert commit 3e255e9 which is to adjust stack size
on qemu_x86 platform for coverage test, but break other
platform's CI test.

Fixes: zephyrproject-rtos#15379.

Signed-off-by: Wentong Wu <[email protected]>
@nashif
Copy link
Member

nashif commented Apr 12, 2019

@mike-scott Actually this change passed the CI test before check in. Anyway I will fix it ASAP.

CI does not run on real HW

nashif pushed a commit that referenced this issue Apr 12, 2019
revert commit 3e255e9 which is to adjust stack size
on qemu_x86 platform for coverage test, but break other
platform's CI test.

Fixes: #15379.

Signed-off-by: Wentong Wu <[email protected]>
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 platform: nRF Nordic nRFx priority: medium Medium impact/importance bug
Projects
None yet
4 participants