monitor_checkStack(): fix pasto introduced in previous change. Spotted by R.Gianassi.

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

diff --git a/bertos/kern/preempt.c b/bertos/kern/preempt.c
index 0cbaf7c902dd2d65a03015cecf2b15804853f17e..9af6c0d1b402a973cc144532c15c2f1ae5d15d2f 100644 (file)
--- a/bertos/kern/preempt.c
+++ b/bertos/kern/preempt.c
@@ -26,8 +26,7 @@
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2001, 2004 Develer S.r.l. (http://www.develer.com/)
- * Copyright 1999, 2000, 2001, 2008 Bernie Innocenti <bernie@codewiz.org>
+ * Copyright 2008 Bernie Innocenti <bernie@codewiz.org>
  * -->
  *
  * \brief Simple realtime multitasking scheduler.
@@ -35,15 +34,15 @@
  *
  * \version $Id: proc.c 1616 2008-08-10 19:41:26Z bernie $
  * \author Bernie Innocenti <bernie@codewiz.org>
- * \author Stefano Fedrigo <aleph@develer.com>
  */
 
 #include "proc_p.h"
 #include "proc.h"
 
+#include <kern/irq.h>
 #include <cpu/frame.h> // CPU_IDLE
-
-#include <unistd.h> // XXX alarm()
+#include <drv/timer.h>
+#include <cfg/module.h>
 
 
 /*
@@ -52,6 +51,8 @@
  */
 uint16_t Quantum;
 
+Timer preempt_timer;
+
 /**
  * Disable preemptive task switching.
  *
@@ -83,8 +84,6 @@ void proc_permit(void)
        --CurrentProcess->forbid_cnt;
 }
 
-static void (*irq_handlers[100])(void); // FIXME
-
 
 void proc_preempt(void)
 {
@@ -92,10 +91,12 @@ void proc_preempt(void)
 
        ATOMIC(LIST_ASSERT_VALID(&ProcReadyList));
 
+       TRACEMSG("hello1");
        IRQ_DISABLE;
        /* Poll on the ready queue for the first ready process */
        while (!(CurrentProcess = (struct Process *)list_remHead(&ProcReadyList)))
        {
+       //TRACEMSG("hello2");
                /*
                 * Make sure we physically reenable interrupts here, no matter what
                 * the current task status is. This is important because if we
@@ -109,24 +110,36 @@ void proc_preempt(void)
                 * are reloaded.
                 */
                IRQ_ENABLE;
-               CPU_IDLE;
+               //FIXME: calls Qt stuff from sighandler! CPU_IDLE;
                MEMORY_BARRIER;
                IRQ_DISABLE;
+       //TRACEMSG("hello3");
        }
        IRQ_ENABLE;
+       TRACEMSG("hello4");
 }
 
-void proc_preempt_timer(void)
+void proc_preempt_timer(UNUSED_ARG(void *, param))
 {
-       // TODO: check Quantum
+       IRQ_DISABLE;
+       if (CurrentProcess)
+       {
+               TRACEMSG("preempting %p:%s", CurrentProcess, CurrentProcess->monitor.name);
+               SCHED_ENQUEUE(CurrentProcess);
+               IRQ_ENABLE;
+               proc_preempt();
+       }
+       IRQ_ENABLE;
 
-       alarm(1);
-       ATOMIC(SCHED_ENQUEUE(CurrentProcess));
-       proc_schedule();
+       timer_setDelay(&preempt_timer, CONFIG_KERN_QUANTUM);
+       timer_add(&preempt_timer);
 }
 
 void proc_schedule(void)
 {
+       TRACE;
+
+       // Will invoke proc_preempt() in interrupt context
        kill(0, SIGUSR1);
 }
 
@@ -137,50 +150,20 @@ void proc_yield(void)
        proc_schedule();
 }
 
-/* signal handler */
-void irq_entry(int signum)
+void proc_entry(void (*user_entry)(void))
 {
-       Process *old_process;
-
-       TRACEMSG("storing %p:%s", CurrentProcess, CurrentProcess->monitor.name);
-       CurrentProcess->leaving = false;
-       getcontext(&CurrentProcess->context);
-       /* We get here in two ways: directly, and after setcontext() below */
-
-       if (CurrentProcess->leaving)
-       {
-               TRACEMSG("leaving to %p:%s", CurrentProcess, CurrentProcess->monitor.name);
-               return;
-       }
-
-       old_process = CurrentProcess;
-
-       irq_handlers[signum]();
-
-       if (old_process != CurrentProcess)
-       {
-               TRACEMSG("launching %p:%s", CurrentProcess, CurrentProcess->monitor.name);
-               CurrentProcess->leaving = true;
-               setcontext(&CurrentProcess->context);
-               /* not reached */
-       }
-
-       TRACEMSG("keeping %p:%s", CurrentProcess, CurrentProcess->monitor.name);
+       user_entry();
+       proc_exit();
 }
 
 void preempt_init(void)
 {
-       struct sigaction act;
-       act.sa_handler = irq_entry;
-       sigemptyset(&act.sa_mask);
-       sigaddset(&act.sa_mask, SIGUSR1);
-       sigaddset(&act.sa_mask, SIGALRM);
-       act.sa_flags = SA_RESTART; /* | SA_SIGINFO; */
-
-       irq_handlers[SIGUSR1] = proc_preempt;
-       irq_handlers[SIGALRM] = proc_preempt_timer;
-       sigaction(SIGUSR1, &act, NULL);
-       sigaction(SIGALRM, &act, NULL);
-
-       alarm(1);
+       MOD_CHECK(irq);
+       MOD_CHECK(timer);
+
+       irq_register(SIGUSR1, proc_preempt);
+
+       timer_setSoftint(&preempt_timer, proc_preempt_timer, NULL);
+       timer_setDelay(&preempt_timer, CONFIG_KERN_QUANTUM);
+       timer_add(&preempt_timer);
 }