[riot-notifications] [RIOT-OS/RIOT] tests/posix_semaphore: unify and increase allowed test4 margin (#11467)

Kaspar Schleiser notifications at github.com
Tue Apr 30 11:46:13 CEST 2019


### Contribution description

The test configured tight margins for how much too long a semaphore wait can take. This is becoming a maintenance burden.

Instead of adding more special cases, this PR changes the allowed margin to 1ms (which is 0.1percent of the 1sec delay). This should be high enough to not trip over slightly inaccurate timers or slow platforms, but high enough to show fundamental problems with the interaction between timers and posix semaphores.

Remember, this should test semaphores, not timer subsystem accuracy.

### Testing procedure

Validate soundness, execute tests on some boards (or let CI do it),

### Issues/PRs references

#11449
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/11467

-- Commit Summary --

  * tests/posix_semaphore: unify and increase allowed test4 margin

-- File Changes --

    M tests/posix_semaphore/main.c (11)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11467.patch
https://github.com/RIOT-OS/RIOT/pull/11467.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11467
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190430/bd650692/attachment.html>


More information about the notifications mailing list