Skip to content
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

Fixes CMSIS v2 test on x86 #30621

Merged
merged 3 commits into from
Dec 17, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions tests/portability/cmsis_rtos_v2/prj.conf
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,4 @@ CONFIG_CMSIS_V2_THREAD_DYNAMIC_MAX_COUNT=10
CONFIG_TIMEOUT_64BIT=n
CONFIG_CMSIS_V2_THREAD_MAX_STACK_SIZE=1024
CONFIG_CMSIS_V2_THREAD_DYNAMIC_STACK_SIZE=1024
CONFIG_MP_NUM_CPUS=1
3 changes: 3 additions & 0 deletions tests/portability/cmsis_rtos_v2/src/event_flags.c
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,9 @@ void test_event_flags_signalled(void)
id1 = osThreadNew(thread1, evt_id, &thread1_attr);
zassert_true(id1 != NULL, "Failed creating thread1");

/* Let id1 run to trigger FLAG1 */
osDelay(2);

id2 = osThreadNew(thread2, evt_id, &thread2_attr);
zassert_true(id2 != NULL, "Failed creating thread2");

Expand Down
4 changes: 1 addition & 3 deletions tests/portability/cmsis_rtos_v2/testcase.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
tests:
portability.cmsis_rtos_v2:
# qemu_x86_64, ehl_crb and up_squared need bigger stack
# but CMSIS limits the stack size, resulting in stack overflow.
platform_exclude: qemu_x86_64 m2gl025_miv up_squared ehl_crb
platform_exclude: m2gl025_miv
tags: cmsis_rtos
min_ram: 32
min_flash: 34