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

2 files changed, 23 insertions, 7 deletions

diff --git a/src/chthreads.c b/src/chthreads.c
index d32f9d44a..902867a41 100644
--- a/src/chthreads.c
+++ b/src/chthreads.c
@@ -59,10 +59,10 @@ Thread *init_thread(Thread *tp, tprio_t prio) {
 }
 
 #if CH_DBG_FILL_THREADS
-static void memfill(uint8_t *p, uint32_t n, uint8_t v) {
+static void memfill(uint8_t *startp, uint8_t *endp, uint8_t v) {
 
-  while (n)
-    *p++ = v, n--;
+  while (startp < endp)
+    *startp++ = v;
 }
 #endif
 
@@ -95,7 +95,12 @@ Thread *chThdInit(void *workspace, size_t wsize,
              (prio <= HIGHPRIO) && (pf != NULL),
              "chThdInit");
 #if CH_DBG_FILL_THREADS
-  memfill(workspace, wsize, MEM_FILL_PATTERN);
+  memfill(workspace,
+		  (uint8_t)workspace + sizeof(Thread),
+		  THREAD_FILL_VALUE);
+  memfill((uint8_t)workspace + sizeof(Thread),
+		  (uint8_t)workspace + wsize
+		  STACK_FILL_VALUE);
 #endif
   SETUP_CONTEXT(workspace, wsize, pf, arg);
   return init_thread(tp, prio);
diff --git a/src/include/debug.h b/src/include/debug.h
index 2efc6a8c7..82e784040 100644
--- a/src/include/debug.h
+++ b/src/include/debug.h
@@ -35,10 +35,21 @@
 #endif

 

 /**

- * @brief Fill value for threads working area in debug mode.

+ * @brief Fill value for thread stack area in debug mode.

  */

-#ifndef MEM_FILL_PATTERN

-#define MEM_FILL_PATTERN 0x55

+#ifndef STACK_FILL_VALUE

+#define STACK_FILL_VALUE 0x55

+#endif

+

+/**

+ * @brief Fill value for thread area in debug mode.

+ * @note The chosen default value is 0xFF in order to make evident which

+ *       thread fields were not initialized when inspecting the memory with

+ *       a debugger. A uninitialized field is not an error in itself but it

+ *       better to know it.

+ */

+#ifndef THREAD_FILL_VALUE

+#define THREAD_FILL_VALUE 0xFF

 #endif

 

 /**