From: Olivier Dion Date: Fri, 14 Jul 2023 16:59:23 +0000 (-0400) Subject: tests/regression/rcutorture: Add wait state X-Git-Url: https://git.lttng.org/?p=userspace-rcu.git;a=commitdiff_plain;h=a6ca497d47f123d2fa49675816d52b7541f3509c tests/regression/rcutorture: Add wait state pthread_cond_wait(3) can have spurious wakeups. Fix this by polling a state associated with the the wait. Change-Id: Iba034cba5f72ad88388d1b90a6093f4ae9f9beb9 Signed-off-by: Olivier Dion Signed-off-by: Mathieu Desnoyers --- diff --git a/tests/regression/rcutorture.h b/tests/regression/rcutorture.h index ead5b40..b3a300c 100644 --- a/tests/regression/rcutorture.h +++ b/tests/regression/rcutorture.h @@ -336,6 +336,7 @@ void *rcu_read_stress_test(void *arg __attribute__((unused))) static pthread_mutex_t call_rcu_test_mutex = PTHREAD_MUTEX_INITIALIZER; static pthread_cond_t call_rcu_test_cond = PTHREAD_COND_INITIALIZER; +static bool call_rcu_wait; static void rcu_update_stress_test_rcu(struct rcu_head *head __attribute__((unused))) @@ -356,6 +357,7 @@ void rcu_update_stress_test_rcu(struct rcu_head *head __attribute__((unused))) strerror(errno)); abort(); } + call_rcu_wait = false; ret = pthread_mutex_unlock(&call_rcu_test_mutex); if (ret) { errno = ret; @@ -437,8 +439,11 @@ void *rcu_update_stress_test(void *arg __attribute__((unused))) * immediately after call_rcu (call_rcu needs * us to be registered RCU readers). */ - ret = pthread_cond_wait(&call_rcu_test_cond, - &call_rcu_test_mutex); + call_rcu_wait = true; + do { + ret = pthread_cond_wait(&call_rcu_test_cond, + &call_rcu_test_mutex); + } while (call_rcu_wait); if (ret) { errno = ret; diag("pthread_cond_signal: %s",