X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fkern%2Fcoop.c;h=c874ca36f8bdaac539f3078c4b70efc5fe6e79c5;hb=6fa2660c8ba089bbf8a1d711f46f882604ff9a6c;hp=4b2e6b88a5c303598391e776789e4007f67e7681;hpb=c0cf049d091313c15e32ab9e521e4dcbc4662698;p=bertos.git

diff --git a/bertos/kern/coop.c b/bertos/kern/coop.c
index 4b2e6b88..c874ca36 100644
--- a/bertos/kern/coop.c
+++ b/bertos/kern/coop.c
@@ -32,7 +32,7 @@
  *
  * \brief Simple cooperative multitasking scheduler.
  *
- * \version $Id: proc.c 1616 2008-08-10 19:41:26Z bernie $
+ * \version $Id$
  * \author Bernie Innocenti <bernie@codewiz.org>
  * \author Stefano Fedrigo <aleph@develer.com>
  */
@@ -56,28 +56,31 @@
  * Saving and restoring the context on the stack is done by a CPU-dependent
  * support routine which usually needs to be written in assembly.
  */
-EXTERN_C void asm_switch_context(cpustack_t **new_sp, cpustack_t **save_sp);
+EXTERN_C void asm_switch_context(cpu_stack_t **new_sp, cpu_stack_t **save_sp);
 
+/**
+ * Define function prototypes exported outside.
+ *
+ * Required to silent gcc "no previous prototype" warnings.
+ */
+void coop_yield(void);
+void coop_switch(void);
 
 /**
  * System scheduler: pass CPU control to the next process in
  * the ready queue.
  */
-void proc_schedule(void)
+static void coop_schedule(void)
 {
-	struct Process *old_process;
-	cpuflags_t flags;
+	cpu_flags_t flags;
 
-	ATOMIC(LIST_ASSERT_VALID(&ProcReadyList));
+	ATOMIC(LIST_ASSERT_VALID(&proc_ready_list));
 	ASSERT_USER_CONTEXT();
-	ASSERT_IRQ_ENABLED();
-
-	/* Remember old process to save its context later */
-	old_process = CurrentProcess;
+	IRQ_ASSERT_ENABLED();
 
 	/* Poll on the ready queue for the first ready process */
 	IRQ_SAVE_DISABLE(flags);
-	while (!(CurrentProcess = (struct Process *)list_remHead(&ProcReadyList)))
+	while (!(current_process = (struct Process *)list_remHead(&proc_ready_list)))
 	{
 		/*
 		 * Make sure we physically reenable interrupts here, no matter what
@@ -86,7 +89,7 @@ void proc_schedule(void)
 		 * process will ever wake up.
 		 *
 		 * During idle-spinning, an interrupt can occur and it may
-		 * modify \p ProcReadyList. To ensure that compiler reload this
+		 * modify \p proc_ready_list. To ensure that compiler reload this
 		 * variable every while cycle we call CPU_MEMORY_BARRIER.
 		 * The memory barrier ensure that all variables used in this context
 		 * are reloaded.
@@ -100,19 +103,27 @@ void proc_schedule(void)
 		IRQ_DISABLE;
 	}
 	IRQ_RESTORE(flags);
+}
+
+void coop_switch(void)
+{
+	/* Remember old process to save its context later */
+	Process * const old_process = current_process;
+
+	coop_schedule();
 
 	/*
 	 * Optimization: don't switch contexts when the active
 	 * process has not changed.
 	 */
-	if (CurrentProcess != old_process)
+	if (current_process != old_process)
 	{
-		cpustack_t *dummy;
+		cpu_stack_t *dummy;
 
 		#if CONFIG_KERN_MONITOR
 			LOG_INFO("Switch from %p(%s) to %p(%s)\n",
-				old_process,    old_process ? old_process->monitor.name : "NONE",
-				CurrentProcess, CurrentProcess->monitor.name);
+				old_process,    proc_name(old_process),
+				current_process, proc_currentName());
 		#endif
 
 		/* Save context of old process and switch to new process. If there is no
@@ -122,19 +133,17 @@ void proc_schedule(void)
 		 * TODO: Instead of physically clearing the process at exit time, a zombie
 		 * list should be created.
 		 */
-		asm_switch_context(&CurrentProcess->stack, old_process ? &old_process->stack : &dummy);
+		asm_switch_context(&current_process->stack, old_process ? &old_process->stack : &dummy);
 	}
 
 	/* This RET resumes the execution on the new process */
 }
 
-
 /**
  * Co-operative context switch
  */
-void proc_yield(void)
+void coop_yield(void)
 {
-	ATOMIC(SCHED_ENQUEUE(CurrentProcess));
-
-	proc_schedule();
+	ATOMIC(SCHED_ENQUEUE(current_process));
+	proc_switch();
 }