X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=kern%2Fmonitor.c;h=f9cda83d29cff15c0fcca0eb7b2a23aef0eb6eb9;hb=f52e46baa21748f30f30a9e725edbf90b420ed24;hp=907487b133a741c15b315ee4fc9cbb7205da0215;hpb=00e009957d70dab15575ab310ed83f0b78cc8656;p=bertos.git

diff --git a/kern/monitor.c b/kern/monitor.c
index 907487b1..f9cda83d 100755
--- a/kern/monitor.c
+++ b/kern/monitor.c
@@ -1,8 +1,8 @@
-/*!
+/**
  * \file
  * <!--
  * Copyright 2004 Develer S.r.l. (http://www.develer.com/)
- * This file is part of DevLib - See devlib/README for information.
+ * This file is part of DevLib - See README.devlib for information.
  * -->
  *
  * \brief Monitor to check for stack overflows
@@ -14,6 +14,24 @@
 
 /*#*
  *#* $Log$
+ *#* Revision 1.9  2006/07/19 12:56:27  bernie
+ *#* Convert to new Doxygen style.
+ *#*
+ *#* Revision 1.8  2006/03/22 09:49:22  bernie
+ *#* Reduce memory usage.
+ *#*
+ *#* Revision 1.7  2006/03/13 02:07:14  bernie
+ *#* Add TODO item.
+ *#*
+ *#* Revision 1.6  2006/02/24 01:17:05  bernie
+ *#* Update for new emulator.
+ *#*
+ *#* Revision 1.5  2005/11/04 16:20:02  bernie
+ *#* Fix reference to README.devlib in header.
+ *#*
+ *#* Revision 1.4  2005/04/11 19:10:28  bernie
+ *#* Include top-level headers from cfg/ subdir.
+ *#*
  *#* Revision 1.3  2004/11/28 23:20:25  bernie
  *#* Remove obsolete INITLIST macro.
  *#*
@@ -39,8 +57,8 @@
 #include <mware/list.h>
 #include <drv/timer.h>
 #include <kern/proc.h>
-#include <macros.h>
-#include <debug.h>
+#include <cfg/macros.h>
+#include <cfg/debug.h>
 
 
 static List MonitorProcs;
@@ -72,10 +90,11 @@ void monitor_rename(Process *proc, const char* name)
 	proc->monitor.name = name;
 }
 
+/* TODO: use containerof() */
 #define MONITOR_NODE_TO_PROCESS(node) \
-	(struct Process*)((char*)(node) - offsetof(struct Process, monitor.link))
+	(struct Process *)((char *)(node) - offsetof(struct Process, monitor.link))
 
-size_t monitor_check_stack(cpustack_t* stack_base, size_t stack_size)
+size_t monitor_checkStack(cpustack_t* stack_base, size_t stack_size)
 {
 	cpustack_t* beg;
 	cpustack_t* cur;
@@ -114,45 +133,43 @@ void monitor_report(void)
 	struct Process* p;
 	int i;
 
-	if (ISLISTEMPTY(&MonitorProcs))
+	if (LIST_EMPTY(&MonitorProcs))
 	{
 		kputs("No stacks registered in the monitor\n");
 		return;
 	}
 
-	kprintf("%-24s    %-6s%-8s%-8s%-8s\n", "Process name", "TCB", "SPbase", "SPsize", "SPfree");
+	kprintf("%-24s%-8s%-8s%-8s%-8s\n", "Process name", "TCB", "SPbase", "SPsize", "SPfree");
 	for (i=0;i<56;i++)
 		kputchar('-');
 	kputchar('\n');
 
-	for (p = MONITOR_NODE_TO_PROCESS(MonitorProcs.head);
+	for (p = MONITOR_NODE_TO_PROCESS(LIST_HEAD(&MonitorProcs));
 		 p->monitor.link.succ;
 		 p = MONITOR_NODE_TO_PROCESS(p->monitor.link.succ))
 	{
-		size_t free = monitor_check_stack(p->monitor.stack_base, p->monitor.stack_size);
-		kprintf("%-24s    %04x    %04x    %4x    %4x\n", p->monitor.name, (uint16_t)p, (uint16_t)p->monitor.stack_base, (uint16_t)p->monitor.stack_size, (uint16_t)free);
+		size_t free = monitor_checkStack(p->monitor.stack_base, p->monitor.stack_size);
+		kprintf("%-24s%8p%8p%8lx%8lx\n",
+			p->monitor.name, p, p->monitor.stack_base, p->monitor.stack_size, free);
 	}
 }
 
 
-static void monitor(void)
+static void NORETURN monitor(void)
 {
-	struct Process* p;
+	struct Process *p;
 
 	while (1)
 	{
-		for (p = MONITOR_NODE_TO_PROCESS(MonitorProcs.head);
+		for (p = MONITOR_NODE_TO_PROCESS(LIST_HEAD(&MonitorProcs));
 			p->monitor.link.succ;
 			p = MONITOR_NODE_TO_PROCESS(p->monitor.link.succ))
 		{
-			size_t free = monitor_check_stack(p->monitor.stack_base, p->monitor.stack_size);
+			size_t free = monitor_checkStack(p->monitor.stack_base, p->monitor.stack_size);
 
 			if (free < 0x20)
-			{
-				kprintf("MONITOR: ***************************************\n");
-				kprintf("MONITOR: WARNING: Free stack for process '%s' is only %04x chars\n", p->monitor.name, free);
-				kprintf("MONITOR: ***************************************\n\n");
-			}
+				kprintf("MONITOR: WARNING: Free stack for process '%s' is only %x chars\n",
+						p->monitor.name, free);
 
 			timer_delay(500);
 		}