From 5a3a608ad919591b88af842b0ce15f4e22790710 Mon Sep 17 00:00:00 2001
From: barthess <barthess@35acf78f-673a-0410-8e92-d51de3d6d3f4>
Date: Tue, 20 Sep 2011 16:00:30 +0000
Subject: Fixed bug 3411207

git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@3360 35acf78f-673a-0410-8e92-d51de3d6d3f4
---
 os/kernel/src/chthreads.c | 1 +
 1 file changed, 1 insertion(+)

(limited to 'os/kernel/src/chthreads.c')

diff --git a/os/kernel/src/chthreads.c b/os/kernel/src/chthreads.c
index 46039561b..73236c850 100644
--- a/os/kernel/src/chthreads.c
+++ b/os/kernel/src/chthreads.c
@@ -353,6 +353,7 @@ void chThdExit(msg_t msg) {
     REG_REMOVE(tp);
 #endif
   chSchGoSleepS(THD_STATE_FINAL);
+  chSysUnlock();
 }
 
 #if CH_USE_WAITEXIT || defined(__DOXYGEN__)
-- 
cgit v1.2.3


From 5463b41d3a7a81da20c88631535c088decd61999 Mon Sep 17 00:00:00 2001
From: gdisirio <gdisirio@35acf78f-673a-0410-8e92-d51de3d6d3f4>
Date: Tue, 20 Sep 2011 17:08:22 +0000
Subject: git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@3362
 35acf78f-673a-0410-8e92-d51de3d6d3f4

---
 os/kernel/src/chthreads.c | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

(limited to 'os/kernel/src/chthreads.c')

diff --git a/os/kernel/src/chthreads.c b/os/kernel/src/chthreads.c
index 73236c850..b2231ef17 100644
--- a/os/kernel/src/chthreads.c
+++ b/os/kernel/src/chthreads.c
@@ -335,6 +335,27 @@ void chThdYield(void) {
  * @api
  */
 void chThdExit(msg_t msg) {
+
+  chSysLock();
+  chThdExitS(msg);
+  /* The thread never returns here.*/
+}
+
+/**
+ * @brief   Terminates the current thread.
+ * @details The thread goes in the @p THD_STATE_FINAL state holding the
+ *          specified exit status code, other threads can retrieve the
+ *          exit status code by invoking the function @p chThdWait().
+ * @post    Eventual code after this function will never be executed,
+ *          this function never returns. The compiler has no way to
+ *          know this so do not assume that the compiler would remove
+ *          the dead code.
+ *
+ * @param[in] msg       thread exit code
+ *
+ * @sclass
+ */
+void chThdExitS(msg_t msg) {
   Thread *tp = currp;
 
   chSysLock();
@@ -353,7 +374,8 @@ void chThdExit(msg_t msg) {
     REG_REMOVE(tp);
 #endif
   chSchGoSleepS(THD_STATE_FINAL);
-  chSysUnlock();
+  /* The thread never returns here.*/
+  chDbgAssert(FALSE, "chThdExitS(), #1", "zombies apocalypse");
 }
 
 #if CH_USE_WAITEXIT || defined(__DOXYGEN__)
-- 
cgit v1.2.3


From c69417e365e3f495588ad7b88e9ee8fc16b3fcdb Mon Sep 17 00:00:00 2001
From: gdisirio <gdisirio@35acf78f-673a-0410-8e92-d51de3d6d3f4>
Date: Tue, 20 Sep 2011 17:10:56 +0000
Subject: Fixed bug 3411207.

git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@3363 35acf78f-673a-0410-8e92-d51de3d6d3f4
---
 os/kernel/src/chthreads.c | 1 -
 1 file changed, 1 deletion(-)

(limited to 'os/kernel/src/chthreads.c')

diff --git a/os/kernel/src/chthreads.c b/os/kernel/src/chthreads.c
index b2231ef17..bf43c6c43 100644
--- a/os/kernel/src/chthreads.c
+++ b/os/kernel/src/chthreads.c
@@ -358,7 +358,6 @@ void chThdExit(msg_t msg) {
 void chThdExitS(msg_t msg) {
   Thread *tp = currp;
 
-  chSysLock();
   tp->p_u.exitcode = msg;
 #if defined(THREAD_EXT_EXIT_HOOK)
   THREAD_EXT_EXIT_HOOK(tp);
-- 
cgit v1.2.3


From c39d08fc2ae9c43f73114e24292520306bddde19 Mon Sep 17 00:00:00 2001
From: gdisirio <gdisirio@35acf78f-673a-0410-8e92-d51de3d6d3f4>
Date: Fri, 23 Sep 2011 15:48:55 +0000
Subject: git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@3384
 35acf78f-673a-0410-8e92-d51de3d6d3f4

---
 os/kernel/src/chthreads.c | 2 --
 1 file changed, 2 deletions(-)

(limited to 'os/kernel/src/chthreads.c')

diff --git a/os/kernel/src/chthreads.c b/os/kernel/src/chthreads.c
index bf43c6c43..7b48f2b00 100644
--- a/os/kernel/src/chthreads.c
+++ b/os/kernel/src/chthreads.c
@@ -32,8 +32,6 @@
  *          area. In this scenario static variables are shared among all
  *          threads while automatic variables are local to the thread.<br>
  *          Operations defined for threads:
- *          - <b>Init</b>, a thread is prepared and put in the suspended
- *            state.
  *          - <b>Create</b>, a thread is started on the specified thread
  *            function. This operation is available in multiple variants,
  *            both static and dynamic.
-- 
cgit v1.2.3