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

Martine Lenders notifications at github.com
Mon May 6 16:14:07 CEST 2019


miri64 commented on this pull request.



> @@ -236,16 +236,16 @@ void test3(void)
     sem_post(&s1);
 }
 
-#ifdef BOARD_NATIVE
-/* native can sometime take more time to respond as it is not real time */
-#define TEST4_TIMEOUT_EXCEEDED_MARGIN (300)
-#elif CPU_FAM_NRF51
-/* nrf51 based boards needs a slightly higher margin value. Using 105us makes
- test4 result more reliable. */
-#define TEST4_TIMEOUT_EXCEEDED_MARGIN (105)
-#else
-#define TEST4_TIMEOUT_EXCEEDED_MARGIN (100)
-#endif /* BOARD_NATIVE */
+/*
+ * Allowed margin for waiting too long.
+ *
+ * Waiting too short is forbidden by POSIX, but is checked elsewhere.

What it is it?

-- 
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#discussion_r281203142
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190506/cc42d8db/attachment-0001.html>


More information about the notifications mailing list