Skip to content

Conversation

@vonhust
Copy link

@vonhust vonhust commented Jul 17, 2018

fix the wrong mpu configuration.

verified by tests/kernel/mem_protection/userspace

Signed-off-by: Wayne Ren wei.ren@synopsys.com

Fixes #8949

fix the wrong mpu configuration.

verified by tests/kernel/mem_protection/userspace

Signed-off-by: Wayne Ren <wei.ren@synopsys.com>
@vonhust vonhust requested a review from ruuddw as a code owner July 17, 2018 06:53
@codecov-io
Copy link

Codecov Report

Merging #8957 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8957   +/-   ##
=======================================
  Coverage   52.25%   52.25%           
=======================================
  Files         196      196           
  Lines       24802    24802           
  Branches     5155     5155           
=======================================
  Hits        12961    12961           
  Misses       9762     9762           
  Partials     2079     2079

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fd3eb6...940ee85. Read the comment docs.

@nashif
Copy link
Member

nashif commented Jul 17, 2018

@vonhust tried this fix, does not resolve the issue for me. How do you run the nsim_sem? looks like we are missing something.

@andrewboie andrewboie requested a review from nashif July 17, 2018 19:43
@vonhust
Copy link
Author

vonhust commented Jul 18, 2018

@nashif could you provide the failed tests/samples, so i can try to reproduce it.
BTW, what version of nsim do you use? I am using the latest version of nsim (nsim 2018.06). For secureshield features, they are added in 2017.

@nashif
Copy link
Member

nashif commented Jul 18, 2018

aha, I have an old version of nSim I guess :(

@nashif nashif merged commit c056340 into zephyrproject-rtos:master Jul 18, 2018
@vonhust vonhust deleted the arc_nsim_fixes branch December 13, 2018 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nsim_sem board does not work

4 participants