Remove the base/advance choice page.

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

diff --git a/bertos/kern/monitor.c b/bertos/kern/monitor.c
index 26e2025b514dab010ec961ae298d0567a3892f5e..edcf7776d9549277bf0aae4af9b7812dee372070 100644 (file)
--- a/bertos/kern/monitor.c
+++ b/bertos/kern/monitor.c
@@ -42,18 +42,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 +118,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 +147,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);
                }