michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: userspace: use 1cpu for some cases

Browse source 
This test spawns a child thread and expects it to complete.
Use one CPU for it. Get rid of the useless k_thread_abort()
call and add a k_yield() to ensure the child does its
thing.

Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>
This commit is contained in:
Andrew Boie 2019-12-18 11:35:28 -08:00 • committed by Anas Nashif
commit 24ff46c634
1 changed files with 5 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

11
tests/kernel/mem_protect/userspace/src/main.c
View file

@ -654,8 +654,7 @@ static void access_other_memdomain(void)
(k_thread_entry_t)shared_mem_thread, NULL,
NULL, NULL, -1, K_USER | K_INHERIT_PERMS, K_NO_WAIT);
k_thread_abort(k_current_get());
k_yield(); /* Let other thread run */
}
@ -1190,14 +1189,14 @@ void test_main(void)
ztest_user_unit_test(pass_user_object),
ztest_user_unit_test(pass_noperms_object),
ztest_user_unit_test(start_kernel_thread),
ztest_user_unit_test(read_other_stack),
ztest_user_unit_test(write_other_stack),
ztest_1cpu_user_unit_test(read_other_stack),
ztest_1cpu_user_unit_test(write_other_stack),
ztest_user_unit_test(revoke_noperms_object),
ztest_user_unit_test(access_after_revoke),
ztest_unit_test(user_mode_enter),
ztest_user_unit_test(write_kobject_user_pipe),
ztest_user_unit_test(read_kobject_user_pipe),
ztest_unit_test(access_other_memdomain),
ztest_1cpu_unit_test(access_other_memdomain),
ztest_unit_test(domain_add_thread_drop_to_user),
ztest_unit_test(domain_add_part_drop_to_user),
ztest_unit_test(domain_remove_part_drop_to_user),
@ -1206,7 +1205,7 @@ void test_main(void)
ztest_unit_test(domain_add_part_context_switch),
ztest_unit_test(domain_remove_part_context_switch),
ztest_unit_test(domain_remove_thread_context_switch),
ztest_unit_test(test_stack_buffer),
ztest_1cpu_unit_test(test_stack_buffer),
ztest_user_unit_test(test_unimplemented_syscall),
ztest_user_unit_test(test_bad_syscall),
ztest_user_unit_test(test_oops_panic),