git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@529 35acf78f-673a-0410-8e92-d51de3d6d3f4

docs/reports/MSP430F1611-0.75.txt

@@ -1,5 +1,5 @@
 ***************************************************************************
-Kernel:   ChibiOS/RT 0.8.0
+Kernel:   ChibiOS/RT 0.8.2
 Compiler: GCC 3.2.3 (MSPGCC)
 Options:  -O2 -fomit-frame-pointer
 Settings: MCLK=DCOCLK 750Khz
@@ -65,11 +65,11 @@ Settings: MCLK=DCOCLK 750Khz
 --- Result: SUCCESS
 ---------------------------------------------------------------------------
 --- Test Case 19 (Benchmark, threads creation/termination, worst case)
---- Score : 1166 threads/S
+--- Score : 1236 threads/S
 --- Result: SUCCESS
 ---------------------------------------------------------------------------
 --- Test Case 20 (Benchmark, threads creation/termination, optimal)
---- Score : 1537 threads/S
+--- Score : 1560 threads/S
 --- Result: SUCCESS
 ---------------------------------------------------------------------------
 --- Test Case 21 (Benchmark, mass reschedulation, 5 threads)
@@ -84,3 +84,5 @@ Settings: MCLK=DCOCLK 750Khz
 --- Score : 5632 timers/S
 --- Result: SUCCESS
 ---------------------------------------------------------------------------
+
+Final result: SUCCESS

readme.txt

@@ -18,7 +18,7 @@
 ./ext/                 - External libraries or other code not part of
                          ChibiOS/RT but used in the demo applications.
 ./test/                - Test code, used by some demos.
-./docs/Doxifile        - Doxigen project file.
+./docs/Doxyfile        - Doxygen project file.
 ./docs/html/index.html - ChibiOS/RT documentation (after running doxigen).
                          The documentation is also available on the project
                          web page: http://chibios.sourceforge.net/
@@ -74,11 +74,14 @@ Win32-MinGW            - ChibiOS/RT simulator and demo into a WIN32 process,
 *****************************************************************************
 
 *** 0.8.2 ***
+- FIX: Included the files that were missing from version 0.8.1 distribution.
 - FIX: Duplicated sections in the documentation removed.
 - FIX: Minor problem in Cortex-M3 and AVR ports when the kernel is compiled
   using G++.
 - NEW: Added chPoolAllocI() and chPoolFreeI() APIs in order to allow the use
   of memory pools from interrupt handlers and timer callbacks.
+- CHANGE: Simplified the code for chThdWait(), it is now both smaller and
+  faster. Added an important usage note to the documentation of this API.
 - CHANGE: The macros WorkingArea(), UserStackSize() and StackAlign() are now
   deprecated and will be removed in version 1.0.0. Use the new equivalents
   WORKING_AREA(), THD_WA_SIZE() and STACK_ALIGN() instead.

src/chthreads.c

@@ -40,7 +40,7 @@ Thread *init_thread(Thread *tp, tprio_t prio) {
   tp->p_mtxlist = NULL;
 #endif
 #ifdef CH_USE_WAITEXIT
-  list_init(&tp->p_waiting);
+  tp->p_waiting = NULL;
 #endif
 #ifdef CH_USE_MESSAGES
   queue_init(&tp->p_msgqueue);
@@ -324,8 +324,10 @@ void chThdExit(msg_t msg) {
   tp->p_exitcode = msg;
   THREAD_EXT_EXIT(tp);
 #ifdef CH_USE_WAITEXIT
-  while (notempty(&tp->p_waiting))
+//  while (notempty(&tp->p_waiting))
-    chSchReadyI(list_remove(&tp->p_waiting));
+//    chSchReadyI(list_remove(&tp->p_waiting));
+  if (tp->p_waiting != NULL)
+    chSchReadyI(tp->p_waiting);
 #endif
 #ifdef CH_USE_EXIT_EVENT
   chEvtBroadcastI(&tp->p_exitesource);
@@ -354,14 +356,21 @@ void chThdExit(msg_t msg) {
  *       must not be used as parameter for further system calls.
  * @note The function is available only if the \p CH_USE_WAITEXIT
  *       option is enabled in \p chconf.h.
+ * @note Only one thread can be waiting for another thread at any time. You
+ *       should imagine the threads as having a reference counter that is set
+ *       to one when the thread is created, chThdWait() decreases the reference
+ *       and the memory is freed when the counter reaches zero. In the current
+ *       implementation there is no real reference counter in the thread
+ *       structure but it is a planned extension.
  */
 msg_t chThdWait(Thread *tp) {
   msg_t msg;
 
   chSysLock();
-  chDbgAssert((tp != NULL) && (tp != currp), "chthreads.c, chThdWait()");
+  chDbgAssert((tp != NULL) && (tp != currp) && (tp->p_waiting != NULL),
+              "chthreads.c, chThdWait()");
   if (tp->p_state != PREXIT) {
-    list_insert(currp, &tp->p_waiting);
+    tp->p_waiting = currp;
     chSchGoSleepS(PRWAIT);
   }
   msg = tp->p_exitcode;
@@ -369,12 +378,8 @@ msg_t chThdWait(Thread *tp) {
   chSysUnlock();
   return msg;
 #else /* CH_USE_DYNAMIC */
-  if (notempty(&tp->p_waiting)) {
-    chSysUnlock();
-    return msg;
-  }
 
-  /* This is the last thread waiting for termination, returning memory.*/
+  /* Returning memory.*/
   tmode_t mode = tp->p_flags & P_MEM_MODE_MASK;
   chSysUnlock();
 

src/include/threads.h

@@ -34,22 +34,22 @@
  */
 struct Thread {
   /** Next \p Thread in the threads list.*/
-  Thread            *p_next;
+  Thread                *p_next;
   /* End of the fields shared with the ThreadsList structure. */
   /** Previous \p Thread in the threads list.*/
-  Thread            *p_prev;
+  Thread                *p_prev;
   /* End of the fields shared with the ThreadsQueue structure. */
   /** The thread priority.*/
-  tprio_t           p_prio;
+  tprio_t               p_prio;
   /* End of the fields shared with the ReadyList structure. */
   /** Thread identifier. */
-  tid_t             p_tid;
+  tid_t                 p_tid;
   /** Current thread state.*/
-  tstate_t          p_state;
+  tstate_t              p_state;
   /** Mode flags. */
-  tmode_t           p_flags;
+  tmode_t               p_flags;
   /** Machine dependent processor context.*/
-  Context           p_ctx;
+  Context               p_ctx;
   /*
    * The following fields are merged in unions because they are all
    * state-specific fields. This trick saves some extra space for each
@@ -57,34 +57,34 @@ struct Thread {
    */
   union {
     /** Thread wakeup code (only valid when exiting the \p PRREADY state).*/
-    msg_t           p_rdymsg;
+    msg_t               p_rdymsg;
     /** The thread exit code (only while in \p PREXIT state).*/
-    msg_t           p_exitcode;
+    msg_t               p_exitcode;
 #ifdef CH_USE_SEMAPHORES
     /** Semaphore where the thread is waiting on (only in \p PRWTSEM state).*/
-    Semaphore       *p_wtsemp;
+    Semaphore           *p_wtsemp;
 #endif
 #ifdef CH_USE_MUTEXES
     /** Mutex where the thread is waiting on (only in \p PRWTMTX state).*/
-    Mutex           *p_wtmtxp;
+    Mutex               *p_wtmtxp;
 #endif
 #ifdef CH_USE_CONDVARS
     /** CondVar where the thread is waiting on (only in \p PRWTCOND state).*/
-    CondVar         *p_wtcondp;
+    CondVar             *p_wtcondp;
 #endif
 #ifdef CH_USE_MESSAGES
     /** Destination thread for message send (only in \p PRSNDMSG state).*/
-    Thread          *p_wtthdp;
+    Thread              *p_wtthdp;
 #endif
 #ifdef CH_USE_EVENTS
     /** Enabled events mask (only while in \p PRWTOREVT or \p PRWTANDEVT
         states). */
-    eventmask_t     p_ewmask;
+    eventmask_t         p_ewmask;
 #endif
 #ifdef CH_USE_TRACE
     /** Kernel object where the thread is waiting on. It is only valid when
         the thread is some sleeping states.*/
-    void            *p_wtobjp;
+    void                *p_wtobjp;
 #endif
   };
   /*
@@ -92,29 +92,29 @@ struct Thread {
    */
 #ifdef CH_USE_WAITEXIT
   /** The list of the threads waiting for this thread termination. */
-  ThreadsList       p_waiting;
+  Thread                *p_waiting;
 #endif
 #ifdef CH_USE_EXIT_EVENT
   /** The thread termination \p EventSource. */
-  EventSource       p_exitesource;
+  EventSource           p_exitesource;
 #endif
 #ifdef CH_USE_MESSAGES
-  ThreadsQueue      p_msgqueue;
+  ThreadsQueue          p_msgqueue;
-  msg_t             p_msg;
+  msg_t                 p_msg;
 #endif
 #ifdef CH_USE_EVENTS
   /** Pending events mask. */
-  eventmask_t       p_epending;
+  eventmask_t           p_epending;
 #endif
 #ifdef CH_USE_MUTEXES
   /** List of mutexes owned by this thread, \p NULL terminated. */
-  Mutex             *p_mtxlist;
+  Mutex                 *p_mtxlist;
   /** Thread's own, non-inherited, priority. */
-  tprio_t           p_realprio;
+  tprio_t               p_realprio;
 #endif
 #if defined(CH_USE_DYNAMIC) && defined(CH_USE_MEMPOOLS)
   /** Memory Pool where the thread workspace is returned. */
-  void              *p_mpool;
+  void                  *p_mpool;
 #endif
 #ifdef CH_USE_THREAD_EXT
   THREAD_EXT_FIELDS