patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 7 Packages Projects Releases Wiki Activity

ztest: rename end-of-test semaphore

Browse source 
It was named 'mutex', which is not representative of what it is used for
since it is not used for mutual exclusion, but rather for signaling.

Change-Id: Icfef0011f890b2546af1686ba6b57e3fc13c6576
Signed-off-by: Benjamin Walsh <walsh.benj@gmail.com>
This commit is contained in:
Benjamin Walsh 2017-02-28 20:28:32 -05:00 committed by Anas Nashif
parent 75e643d532
commit 67b816d74c
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
tests/ztest/src/ztest.c
View file

@ -131,18 +131,18 @@ out:
static char __stack thread_stack[CONFIG_ZTEST_STACKSIZE];
static int test_result;
static struct k_sem mutex;
static struct k_sem test_end_signal;
void ztest_test_fail(void)
{
test_result = -1;
k_sem_give(&mutex);
k_sem_give(&test_end_signal);
k_thread_abort(k_current_get());
}
static void init_testing(void)
{
k_sem_init(&mutex, 0, 1);
k_sem_init(&test_end_signal, 0, 1);
}
static void test_cb(void *a, void *dummy2, void *dummy)
@ -156,7 +156,7 @@ static void test_cb(void *a, void *dummy2, void *dummy)
run_test_functions(test);
test_result = 0;
k_sem_give(&mutex);
k_sem_give(&test_end_signal);
}
static int run_test(struct unit_test *test)
@ -180,7 +180,7 @@ static int run_test(struct unit_test *test)
* but has _not_ gone back to _thread_entry() and completed it's "abort
* phase": this will corrupt the kernel ready queue.
*/
k_sem_take(&mutex, K_FOREVER);
k_sem_take(&test_end_signal, K_FOREVER);
if (test_result) {
ret = TC_FAIL;
}