Skip to content

tests: getnameinfo runs with user mode disabled #15228

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.

Sign up for GitHub

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

Jump to bottom
Closed
andrewboie opened this issue Apr 5, 2019 · 0 comments · Fixed by #15230
Closed

tests: getnameinfo runs with user mode disabled #15228

andrewboie opened this issue Apr 5, 2019 · 0 comments · Fixed by #15230
Assignees
@andrewboie
Labels
bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@andrewboie
Copy link
Contributor

tests/net/socket/getnameinfo has CONFIG_TEST_USERSPACE=n set in its prj.conf

This is despite the fact that it tries to run the test in user mode, and works just fine.
@andrewboie andrewboie added the bug The issue is a bug, or the PR is fixing a bug label Apr 5, 2019
@andrewboie andrewboie self-assigned this Apr 5, 2019
@andrewboie andrewboie added the priority: low Low impact/importance bug label Apr 5, 2019
@andrewboie andrewboie mentioned this issue Apr 5, 2019
Merged
andrewboie pushed a commit to andrewboie/zephyr that referenced this issue Apr 5, 2019
tests: net: getnameinfo: run in user mode
e8ad181 
The test is designed to run in user mode, but was disabled
for some reason.

Re-enable, and add mps2_an385 to the whitelist as this is
the QEMU target that emulates an MPU.

Fixes: zephyrproject-rtos#15228

Signed-off-by: Andrew Boie <[email protected]>
nashif pushed a commit that referenced this issue Apr 6, 2019
@nashif
tests: net: getnameinfo: run in user mode
1d0369f 
The test is designed to run in user mode, but was disabled
for some reason.

Re-enable, and add mps2_an385 to the whitelist as this is
the QEMU target that emulates an MPU.

Fixes: #15228

Signed-off-by: Andrew Boie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrewboie andrewboie

Labels
bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
1 participant
@andrewboie