X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fkern%2Fmonitor.c;h=8da306771b927778a3ff8b3823554259d2b4b339;hb=32d1445272120a254d77ce8d1af1f527da7a2c17;hp=1263401a3a807ffc1473b804bf0345bcda003236;hpb=444b1c1d891a14e9b70ba7043d1425ce41345675;p=bertos.git

diff --git a/bertos/kern/monitor.c b/bertos/kern/monitor.c
index 1263401a..8da30677 100644
--- a/bertos/kern/monitor.c
+++ b/bertos/kern/monitor.c
@@ -42,17 +42,20 @@
 #if CONFIG_KERN_MONITOR
 
 #include "proc_p.h"
-#include <mware/list.h>
-#include <drv/timer.h>
-#include <kern/proc.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 */
 
 /* Access to this list must be protected against the scheduler */
 static List MonitorProcs;
 
-
 void monitor_init(void)
 {
 	LIST_INIT(&MonitorProcs);
@@ -77,21 +80,23 @@ void monitor_rename(Process *proc, const char *name)
 	proc->monitor.name = name;
 }
 
-size_t monitor_checkStack(cpustack_t *stack_base, size_t stack_size)
+size_t monitor_checkStack(cpu_stack_t *stack_base, size_t stack_size)
 {
-	cpustack_t *beg;
-	cpustack_t *cur;
-	cpustack_t *end;
+	cpu_stack_t *beg;
+	cpu_stack_t *cur;
+	cpu_stack_t *end;
+	int inc;
 	size_t sp_free;
 
+
 	beg = stack_base;
-	end = stack_base + stack_size / sizeof(cpustack_t) - 1;
+	end = stack_base + stack_size / sizeof(cpu_stack_t);
+	inc = +1;
 
 	if (CPU_STACK_GROWS_UPWARD)
 	{
-		cur = beg;
-		beg = end;
-		end = cur;
+		SWAP(beg, end);
+		inc = -1;
 	}
 
 	cur = beg;
@@ -100,13 +105,10 @@ size_t monitor_checkStack(cpustack_t *stack_base, size_t stack_size)
 		if (*cur != CONFIG_KERN_STACKFILLCODE)
 			break;
 
-		if (CPU_STACK_GROWS_UPWARD)
-			cur--;
-		else
-			cur++;
+		cur += inc;
 	}
 
-	sp_free = ABS(cur - beg) * sizeof(cpustack_t);
+	sp_free = ABS(cur - beg) * sizeof(cpu_stack_t);
 	return sp_free;
 }
 
@@ -116,18 +118,18 @@ void monitor_report(void)
 	Node *node;
 	int i;
 
-	kprintf("%-24s%-8s%-8s%-8s%-8s\n", "Process name", "TCB", "SPbase", "SPsize", "SPfree");
-	for (i = 0; i<56; 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);
 		size_t free = monitor_checkStack(p->stack_base, p->stack_size);
-		kprintf("%-24s%-8p%-8p%-8lu%-8lu\n",
-			p->monitor.name, p, p->stack_base, p->stack_size, free);
+		kprintf("%-9p%-9p%-9zu%-9zu%s\n",
+			p, p->stack_base, p->stack_size, free, p->monitor.name);
 	}
 	proc_permit();
 }
@@ -135,7 +137,6 @@ void monitor_report(void)
 
 static void NORETURN monitor(void)
 {
-	Process *p;
 	Node *node;
 
 	for (;;)
@@ -143,7 +144,7 @@ static void NORETURN monitor(void)
 		proc_forbid();
 		FOREACH_NODE(node, &MonitorProcs)
 		{
-			p = containerof(node, Process, monitor.link);
+			Process *p = containerof(node, Process, monitor.link);
 			size_t free = monitor_checkStack(p->stack_base, p->stack_size);
 
 			if (free < 0x20)
@@ -157,10 +158,10 @@ static void NORETURN monitor(void)
 	}
 }
 
-
-void monitor_start(size_t stacksize, cpustack_t *stack)
+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 */