<div dir="ltr"><div>Thanks, didn't notice it in man pages before... I'll use nanosleep.<br><br>Thanks,<br></div>Marek<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 10 November 2014 13:16, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, 7 Nov 2014 22:08:43 +0200<br>
Giulio Camuffo <<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>> wrote:<br>
<br>
> Looks good, they fail as expected.<br>
><br>
> Reviewed-by: Giulio Camuffo <<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>><br>
><br>
> 2014-09-24 15:37 GMT+03:00 Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>>:<br>
> > sanity test if timeouts work.<br>
> ><br>
> > Signed-off-by: Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>><br>
> > ---<br>
> >  tests/sanity-test.c | 25 +++++++++++++++++++++++++<br>
> >  1 file changed, 25 insertions(+)<br>
> ><br>
> > diff --git a/tests/sanity-test.c b/tests/sanity-test.c<br>
> > index 46f4f85..5a03e7c 100644<br>
> > --- a/tests/sanity-test.c<br>
> > +++ b/tests/sanity-test.c<br>
> > @@ -29,6 +29,9 @@<br>
> >  #include "test-runner.h"<br>
> >  #include "wayland-util.h"<br>
> ><br>
> > +#define WL_HIDE_DEPRECATED<br>
> > +#include "test-compositor.h"<br>
> > +<br>
> >  extern int leak_check_enabled;<br>
> ><br>
> >  TEST(empty)<br>
> > @@ -125,3 +128,25 @@ TEST(sanity_fd_exec)<br>
> ><br>
> >         exec_fd_leak_check(nr_fds + 2);<br>
> >  }<br>
> > +<br>
> > +FAIL_TEST(timeout_tst)<br>
> > +{<br>
> > +       test_set_timeout(1);<br>
> > +       sleep(2);<br>
> > +}<br>
> > +<br>
> > +static void<br>
> > +client_timeout(void)<br>
> > +{<br>
> > +       test_set_timeout(1);<br>
> > +       sleep(2);<br>
> > +}<br>
> > +<br>
> > +/* test timeout with test-compositor */<br>
> > +FAIL_TEST(tc_timeout_tst)<br>
> > +{<br>
> > +       struct display *d = display_create();<br>
> > +       client_create(d, client_timeout);<br>
> > +       display_run(d);<br>
> > +       display_destroy(d);<br>
> > +}<br>
<br>
</div></div>Unfortunately, this is not guaranteed. 'man alarm' says:<br>
"sleep(3) may be implemented using SIGALRM; mixing calls to alarm()<br>
and sleep(3) is a bad idea."<br>
<br>
I suppose we should use nanosleep()?<br>
Like we replaced usleep() with nanosleep() too.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div>