* [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time)
@ 2019-01-14 10:46 Alex Bennée
2019-01-14 13:50 ` Peter Maydell
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Alex Bennée @ 2019-01-14 10:46 UTC (permalink / raw)
To: qemu-devel; +Cc: cota, ehabkost, pbonzini, Alex Bennée
Relying on sleep to always return having slept isn't safe as a signal
may have occurred. If signals are constantly incoming the program will
never reach it's termination condition. This is believed to be the
mechanism causing time outs for qht-test in Travis.
The glib g_usleep() deals with all of this for us so lets use it instead.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/atomic64-bench.c | 6 ++----
tests/atomic_add-bench.c | 6 ++----
tests/qht-bench.c | 6 ++----
3 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/tests/atomic64-bench.c b/tests/atomic64-bench.c
index 71692560ed..121a8c14f4 100644
--- a/tests/atomic64-bench.c
+++ b/tests/atomic64-bench.c
@@ -74,16 +74,14 @@ static void *thread_func(void *arg)
static void run_test(void)
{
- unsigned int remaining;
unsigned int i;
while (atomic_read(&n_ready_threads) != n_threads) {
cpu_relax();
}
+
atomic_set(&test_start, true);
- do {
- remaining = sleep(duration);
- } while (remaining);
+ g_usleep(duration * G_USEC_PER_SEC);
atomic_set(&test_stop, true);
for (i = 0; i < n_threads; i++) {
diff --git a/tests/atomic_add-bench.c b/tests/atomic_add-bench.c
index 2f6c72f63a..5666f6bbff 100644
--- a/tests/atomic_add-bench.c
+++ b/tests/atomic_add-bench.c
@@ -76,16 +76,14 @@ static void *thread_func(void *arg)
static void run_test(void)
{
- unsigned int remaining;
unsigned int i;
while (atomic_read(&n_ready_threads) != n_threads) {
cpu_relax();
}
+
atomic_set(&test_start, true);
- do {
- remaining = sleep(duration);
- } while (remaining);
+ g_usleep(duration * G_USEC_PER_SEC);
atomic_set(&test_stop, true);
for (i = 0; i < n_threads; i++) {
diff --git a/tests/qht-bench.c b/tests/qht-bench.c
index ab4e708180..e3b512f26f 100644
--- a/tests/qht-bench.c
+++ b/tests/qht-bench.c
@@ -398,16 +398,14 @@ static void pr_stats(void)
static void run_test(void)
{
- unsigned int remaining;
int i;
while (atomic_read(&n_ready_threads) != n_rw_threads + n_rz_threads) {
cpu_relax();
}
+
atomic_set(&test_start, true);
- do {
- remaining = sleep(duration);
- } while (remaining);
+ g_usleep(duration * G_USEC_PER_SEC);
atomic_set(&test_stop, true);
for (i = 0; i < n_rw_threads; i++) {
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time)
2019-01-14 10:46 [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time) Alex Bennée
@ 2019-01-14 13:50 ` Peter Maydell
2019-01-14 14:13 ` Emilio G. Cota
2019-01-15 21:22 ` Richard Henderson
2 siblings, 0 replies; 4+ messages in thread
From: Peter Maydell @ 2019-01-14 13:50 UTC (permalink / raw)
To: Alex Bennée
Cc: QEMU Developers, Paolo Bonzini, Emilio G. Cota, Eduardo Habkost
On Mon, 14 Jan 2019 at 11:07, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Relying on sleep to always return having slept isn't safe as a signal
> may have occurred. If signals are constantly incoming the program will
> never reach it's termination condition. This is believed to be the
> mechanism causing time outs for qht-test in Travis.
>
> The glib g_usleep() deals with all of this for us so lets use it instead.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/atomic64-bench.c | 6 ++----
> tests/atomic_add-bench.c | 6 ++----
> tests/qht-bench.c | 6 ++----
> 3 files changed, 6 insertions(+), 12 deletions(-)
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
thanks
-- PMM
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time)
2019-01-14 10:46 [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time) Alex Bennée
2019-01-14 13:50 ` Peter Maydell
@ 2019-01-14 14:13 ` Emilio G. Cota
2019-01-15 21:22 ` Richard Henderson
2 siblings, 0 replies; 4+ messages in thread
From: Emilio G. Cota @ 2019-01-14 14:13 UTC (permalink / raw)
To: Alex Bennée; +Cc: qemu-devel, ehabkost, pbonzini
On Mon, Jan 14, 2019 at 10:46:41 +0000, Alex Bennée wrote:
> Relying on sleep to always return having slept isn't safe as a signal
> may have occurred. If signals are constantly incoming the program will
> never reach it's termination condition. This is believed to be the
> mechanism causing time outs for qht-test in Travis.
>
> The glib g_usleep() deals with all of this for us so lets use it instead.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Emilio G. Cota <cota@braap.org>
Thanks,
E.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time)
2019-01-14 10:46 [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time) Alex Bennée
2019-01-14 13:50 ` Peter Maydell
2019-01-14 14:13 ` Emilio G. Cota
@ 2019-01-15 21:22 ` Richard Henderson
2 siblings, 0 replies; 4+ messages in thread
From: Richard Henderson @ 2019-01-15 21:22 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, cota, ehabkost
On 1/14/19 9:46 PM, Alex Bennée wrote:
> Relying on sleep to always return having slept isn't safe as a signal
> may have occurred. If signals are constantly incoming the program will
> never reach it's termination condition. This is believed to be the
> mechanism causing time outs for qht-test in Travis.
>
> The glib g_usleep() deals with all of this for us so lets use it instead.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/atomic64-bench.c | 6 ++----
> tests/atomic_add-bench.c | 6 ++----
> tests/qht-bench.c | 6 ++----
> 3 files changed, 6 insertions(+), 12 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-01-15 21:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-14 10:46 [Qemu-devel] [RFC PATCH] tests: use g_usleep instead of rem = sleep(time) Alex Bennée
2019-01-14 13:50 ` Peter Maydell
2019-01-14 14:13 ` Emilio G. Cota
2019-01-15 21:22 ` Richard Henderson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.