<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 7 November 2014 21:26, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm a bit confused here... you remove the alarm() calls but not the<br>
signal handler or the sigaction setting.</blockquote><div><br></div><div>The alarm worked with default action - that is kill the process. There were no signal handler or sigaction setting before.<br></div><div>When test failed, it was due to SIGALRM and nothing else.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> On the other hand, in the<br>
patch introducing test_set_timeout() you add a new SIGALRM handler...<br>
Am i missing something?<br></blockquote><div><br></div><div>So now it should be all OK with the handlers :)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
--<br>
Giulio<br>
<div><div class="h5"><br>
<br>
2014-09-24 15:37 GMT+03:00 Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>>:<br>
> replace call to alarm() with test_set_timeout()<br>
><br>
> Signed-off-by: Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>><br>
> ---<br>
>  tests/display-test.c | 8 ++++----<br>
>  1 file changed, 4 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/tests/display-test.c b/tests/display-test.c<br>
> index a1e45b1..8435d60 100644<br>
> --- a/tests/display-test.c<br>
> +++ b/tests/display-test.c<br>
> @@ -425,7 +425,7 @@ threading_post_err(void)<br>
>         /* kill test in 3 seconds. This should be enough time for the<br>
>          * thread to exit if it's not blocking. If everything is OK, than<br>
>          * the thread was woken up and the test will end before the SIGALRM */<br>
> -       alarm(3);<br>
> +       test_set_timeout(3);<br>
>         pthread_join(thread, NULL);<br>
><br>
>         wl_proxy_destroy((struct wl_proxy *) c->tc);<br>
> @@ -477,7 +477,7 @@ threading_cancel_read(void)<br>
>         /* kill test in 3 seconds. This should be enough time for the<br>
>          * thread to exit if it's not blocking. If everything is OK, than<br>
>          * the thread was woken up and the test will end before the SIGALRM */<br>
> -       alarm(3);<br>
> +       test_set_timeout(3);<br>
>         pthread_join(th1, NULL);<br>
>         pthread_join(th2, NULL);<br>
>         pthread_join(th3, NULL);<br>
> @@ -516,7 +516,7 @@ threading_read_eagain(void)<br>
>          * set it - check if we're testing the right case */<br>
>         assert(errno == EAGAIN);<br>
><br>
> -       alarm(3);<br>
> +       test_set_timeout(3);<br>
>         pthread_join(th1, NULL);<br>
>         pthread_join(th2, NULL);<br>
>         pthread_join(th3, NULL);<br>
> @@ -577,7 +577,7 @@ threading_read_after_error(void)<br>
>         assert(wl_display_read_events(c->wl_display) == -1);<br>
><br>
>         /* kill test in 3 seconds */<br>
> -       alarm(3);<br>
> +       test_set_timeout(3);<br>
>         pthread_join(thread, NULL);<br>
><br>
>         wl_proxy_destroy((struct wl_proxy *) c->tc);<br>
> --<br>
> 1.9.3<br>
><br>
</div></div>> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</blockquote></div><br></div><div class="gmail_extra">Thanks,<br>Marek<br></div></div>