chiark / gitweb /
barrier: fix race in test-code
authorDavid Herrmann <dh.herrmann@gmail.com>
Tue, 29 Jul 2014 17:14:23 +0000 (19:14 +0200)
committerDavid Herrmann <dh.herrmann@gmail.com>
Tue, 29 Jul 2014 17:19:49 +0000 (19:19 +0200)
commit359017c1aec9596e9d3c4c19bd62d34a6f3366b8
tree3a10629622bcdd5be8269ceaf4035a8aba9a0f3f
parentf22bbd11dbfae4b7dd0606f6981d38d893ed4c1e
barrier: fix race in test-code

The barrier_wait_next_twice* test-cases run:
  Parent:                             Child:
    set_alarm(10)                       sleep_for(1);
    ...                                 set_alarm(1);
    sleep_for(2)                        ...

Therefore, the parent exits after 2+ periods, the client's alarm fires
after 2+ periods. This race turns out to be lost by the child on other
machines, so avoid it by increasing the parent's sleep-interval to 4. This
way, the client has 2 periods to run the barrier test, which is far more
than enough.
src/test/test-barrier.c