Fixed bug 3074984.

git-svn-id: svn://svn.code.sf.net/p/chibios/svn/branches/stable_2.0.x@2192 35acf78f-673a-0410-8e92-d51de3d6d3f4

os/kernel/src/chschd.c

@@ -121,8 +121,12 @@ void chSchGoSleepS(tstate_t newstate) {
 static void wakeup(void *p) {
   Thread *tp = (Thread *)p;
 
-#if CH_USE_SEMAPHORES || (CH_USE_CONDVARS && CH_USE_CONDVARS_TIMEOUT)
   switch (tp->p_state) {
+  case THD_STATE_READY:
+    /* Handling the special case where the thread has been made ready by
+       another thread with higher priority.*/
+    return;
+#if CH_USE_SEMAPHORES || (CH_USE_CONDVARS && CH_USE_CONDVARS_TIMEOUT)
 #if CH_USE_SEMAPHORES
   case THD_STATE_WTSEM:
     chSemFastSignalI((Semaphore *)tp->p_u.wtobjp);
@@ -133,9 +137,8 @@ static void wakeup(void *p) {
 #endif
     /* States requiring dequeuing.*/
     dequeue(tp);
-  }
 #endif
-  /* Done this way in order to allow a tail call.*/
+  }
   tp->p_u.rdymsg = RDY_TIMEOUT;
   chSchReadyI(tp);
 }

readme.txt

@@ -59,6 +59,8 @@
 *****************************************************************************
 
 *** 2.0.5 ***
+- FIX: Fixed race condition in function chSchGoSleepTimeoutS, thanks Balázs
+  (bug 3074984).
 - FIX: Fixed race condition in threads creation (bug 3069854).
 - FIX: Fixed broken CH_DBG_ENABLE_STACK_CHECK option in legacy CM3 port (bug
   3064274).