X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fkern%2Fproc_p.h;h=6a79eec30d6048194f359f40458e3d8943b39e2d;hb=b812fafcc11015af6fe0ef2b1b0bd81e19cf6eff;hp=77fca314f0719f8ae9548d39cd899a04afc14545;hpb=11da44352fcd4d33b173c976db45293ac1a87cca;p=bertos.git

diff --git a/bertos/kern/proc_p.h b/bertos/kern/proc_p.h
index 77fca314..6a79eec3 100644
--- a/bertos/kern/proc_p.h
+++ b/bertos/kern/proc_p.h
@@ -27,21 +27,20 @@
  * the GNU General Public License.
  *
  * Copyright 2001, 2004 Develer S.r.l. (http://www.develer.com/)
- * Copyright 1999, 2000, 2001 Bernardo Innocenti <bernie@develer.com>
+ * Copyright 1999, 2000, 2001 Bernie Innocenti <bernie@codewiz.org>
  *
  * -->
  *
  * \brief Internal scheduler structures and definitions for processes.
  *
  * \version $Id$
- *
- * \author Bernardo Innocenti <bernie@develer.com>
+ * \author Bernie Innocenti <bernie@codewiz.org>
  */
 
 #ifndef KERN_PROC_P_H
 #define KERN_PROC_P_H
 
-#include <cfg/cfg_kern.h>
+#include "cfg/cfg_kern.h"
 #include <cfg/compiler.h>
 
 #include <cpu/types.h>        /* for cpu_stack_t */
@@ -65,6 +64,9 @@ typedef struct Process
 
 #if CONFIG_KERN_HEAP
 	uint16_t     flags;       /**< Flags */
+#endif
+
+#if CONFIG_KERN_HEAP | CONFIG_KERN_MONITOR | (ARCH & ARCH_EMUL)
 	cpustack_t  *stack_base;  /**< Base of process stack */
 	size_t       stack_size;  /**< Size of process stack */
 #endif
@@ -74,8 +76,6 @@ typedef struct Process
 	{
 		Node        link;
 		const char *name;
-		cpustack_t *stack_base;
-		size_t      stack_size;
 	} monitor;
 #endif
 
@@ -93,11 +93,24 @@ typedef struct Process
 /** Track running processes. */
 extern REGISTER Process	*CurrentProcess;
 
-/** Track ready processes. */
+/**
+ * Track ready processes.
+ *
+ * Access to this list must be protected with a proc_forbid() / proc_premit()
+ * pair, or with SCHED_ATOMIC()
+ */
 extern REGISTER List     ProcReadyList;
 
 
-/** Enqueue a task in the ready list. */
+/**
+ * Enqueue a task in the ready list.
+ *
+ * Always use this macro to instert a process in the ready list, as its
+ * might vary to implement a different scheduling algorithms.
+ *
+ * \note This macro is *NOT* protected against the scheduler.  Access to
+ *       this list must be performed with interrupts disabled.
+ */
 #define SCHED_ENQUEUE(proc)  ADDTAIL(&ProcReadyList, &(proc)->link)
 
 /** Schedule to another process *without* adding the current to the ready list. */
@@ -108,14 +121,13 @@ void proc_schedule(void);
 	void monitor_init(void);
 
 	/** Register a process into the monitor */
-	void monitor_add(Process *proc, const char *name, cpustack_t *stack, size_t stacksize);
+	void monitor_add(Process *proc, const char *name);
 
 	/** Unregister a process from the monitor */
 	void monitor_remove(Process *proc);
 
 	/** Rename a process */
-	void monitor_rename(Process *proc, const char* name);
+	void monitor_rename(Process *proc, const char *name);
 #endif /* CONFIG_KERN_MONITOR */
 
 #endif /* KERN_PROC_P_H */
-