Update preset.

[bertos.git] / bertos / kern / monitor.c

diff --git a/bertos/kern/monitor.c b/bertos/kern/monitor.c
index 5759c5334b2418c437bebbd21a8c2479cc7c64e7..d0bf4a823b1bf534e17e218e262a38bd70c51a11 100644 (file)
--- a/bertos/kern/monitor.c
+++ b/bertos/kern/monitor.c
@@ -32,7 +32,6 @@
  *
  * \brief Monitor to check for stack overflows
  *
- * \version $Id$
  * \author Giovanni Bajo <rasky@develer.com>
  */
 
@@ -42,18 +41,20 @@
 #if CONFIG_KERN_MONITOR
 
 #include "proc_p.h"
+#include <cfg/macros.h>
+#include <cfg/debug.h>
+
 #include <struct/list.h>
+
 #include <drv/timer.h>
+
 #include <kern/proc.h>
-#include <cpu/frame.h> /* CPU_STACK_GROWS_UPWARD */
-#include <cfg/macros.h>
-#include <cfg/debug.h>
 
+#include <cpu/frame.h> /* CPU_STACK_GROWS_UPWARD */
 
 /* Access to this list must be protected against the scheduler */
 static List MonitorProcs;
 
-
 void monitor_init(void)
 {
        LIST_INIT(&MonitorProcs);
@@ -116,12 +117,12 @@ void monitor_report(void)
        Node *node;
        int i;
 
+       proc_forbid();
        kprintf("%-9s%-9s%-9s%-9s%s\n", "TCB", "SPbase", "SPsize", "SPfree", "Name");
        for (i = 0; i < 56; i++)
                kputchar('-');
        kputchar('\n');
 
-       proc_forbid();
        FOREACH_NODE(node, &MonitorProcs)
        {
                Process *p = containerof(node, Process, monitor.link);
@@ -145,7 +146,7 @@ static void NORETURN monitor(void)
                        Process *p = containerof(node, Process, monitor.link);
                        size_t free = monitor_checkStack(p->stack_base, p->stack_size);
 
-                       if (free < 0x20)
+                       if (p->stack_base && free < 0x20)
                                kprintf("MONITOR: Free stack of process '%s' is only %u chars\n",
                                                p->monitor.name, (unsigned int)free);
                }
@@ -156,10 +157,10 @@ static void NORETURN monitor(void)
        }
 }
 
-
 void monitor_start(size_t stacksize, cpu_stack_t *stack)
 {
-       proc_new(monitor, NULL, stacksize, stack);
+       struct Process *p = proc_new(monitor, NULL, stacksize, stack);
+       proc_setPri(p, -10);
 }
 
 #endif /* CONFIG_KERN_MONITOR */