irq: more emancipation from proc/preempt
authorbernie <bernie@38d2e660-2303-0410-9eaa-f027e97ec537>
Sun, 17 Aug 2008 15:49:48 +0000 (15:49 +0000)
committerbernie <bernie@38d2e660-2303-0410-9eaa-f027e97ec537>
Sun, 17 Aug 2008 15:49:48 +0000 (15:49 +0000)
git-svn-id: https://src.develer.com/svnoss/bertos/trunk@1651 38d2e660-2303-0410-9eaa-f027e97ec537

app/demo/demo.c
app/demo/demo.mk
bertos/kern/irq.c [new file with mode: 0644]
bertos/kern/irq.h [new file with mode: 0644]
bertos/kern/preempt.c
bertos/kern/proc.c
bertos/kern/proc.h

index ed64a12d404f78ac6ef3e2703955d8da9c5e18fc..86d8356e92720777c827d31dd7cae9919a9d26c5 100644 (file)
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2006 Develer S.r.l. (http://www.develer.com/)
- *
+ * Copyright 2006, 2008 Develer S.r.l. (http://www.develer.com/)
  * -->
  *
- * \version $Id: demo.c 18242 2007-10-08 17:35:23Z marco $
+ * \brief Multifunction system test for BeRTOS modules.
  *
+ * \version $Id: demo.c 18242 2007-10-08 17:35:23Z marco $
  * \author Bernie Innocenti <bernie@codewiz.org>
- *
- * \brief Windowing system test.
  */
 
 #include <cfg/macros.h>
 
 #include <emul/emul.h>
 
+#include <kern/irq.h>
 #include <kern/proc.h>
 #include <kern/msg.h>
 
@@ -294,6 +293,7 @@ static struct Menu main_menu = { main_items, "Main Menu", MF_STICKY, &lcd_bitmap
 int main(int argc, char *argv[])
 {
        emul_init(&argc, argv);
+       irq_init();
        timer_init();
        buz_init();
        kbd_init();
index 672de108a019f3bdda2919b8eb646510343bb198..51b71bd03237a7954254365c99c6a67a71640020 100644 (file)
@@ -18,8 +18,8 @@ demo_DEBUG = 1
 TRG += demo
 
 # FIXME: we want to use g++ for C source too
-CC = g++-4.1
-CXX = g++-4.1
+CC = g++-4.3
+CXX = g++-4.3
 
 demo_CXXSRC = \
        bertos/emul/emul.cpp \
@@ -52,9 +52,10 @@ demo_CSRC = \
        bertos/mware/observer.c \
        bertos/mware/resource.c \
        bertos/mware/sprintf.c \
+       bertos/kern/irq.c \
+       bertos/kern/preempt.c \
        bertos/kern/proc.c \
        bertos/kern/proc_test.c \
-       bertos/kern/preempt.c \
        bertos/kern/sem.c \
        bertos/kern/signal.c \
        bertos/kern/monitor.c \
diff --git a/bertos/kern/irq.c b/bertos/kern/irq.c
new file mode 100644 (file)
index 0000000..23d19af
--- /dev/null
@@ -0,0 +1,105 @@
+/**
+ * \file
+ * <!--
+ * This file is part of BeRTOS.
+ *
+ * Bertos is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * As a special exception, you may use this file as part of a free software
+ * library without restriction.  Specifically, if other files instantiate
+ * templates or use macros or inline functions from this file, or you compile
+ * this file and link it with other files to produce an executable, this
+ * file does not by itself cause the resulting executable to be covered by
+ * the GNU General Public License.  This exception does not however
+ * invalidate any other reasons why the executable file might be covered by
+ * the GNU General Public License.
+ *
+ * Copyright 2008 Bernie Innocenti <bernie@codewiz.org>
+ * -->
+ *
+ * \brief Process scheduler (public interface).
+ *
+ * \version $Id: proc.h 1646 2008-08-17 13:49:48Z bernie $
+ * \author Bernie Innocenti <bernie@codewiz.org>
+ */
+#include "irq.h"
+
+#include <cfg/module.h>
+#include <kern/proc_p.h>
+
+#include <cfg/cfg_kern.h>
+
+#include <unistd.h> // FIXME: move POSIX stuff to irq_posix.h
+
+
+MOD_DEFINE(irq)
+
+// FIXME
+static void (*irq_handlers[100])(void);
+
+/* signal handler */
+void irq_entry(int signum)
+{
+       Process * const old_process = CurrentProcess;
+
+       irq_handlers[signum]();
+
+#if CONFIG_KERN_PREEMPT
+       if (!CurrentProcess)
+       {
+               TRACEMSG("no runnable processes!");
+               IRQ_ENABLE;
+               pause();
+       }
+       else
+       {
+               if (old_process != CurrentProcess)
+               {
+                       TRACEMSG("switching from %p:%s to %p:%s",
+                               old_process, old_process ? old_process->monitor.name : "-",
+                               CurrentProcess, CurrentProcess->monitor.name);
+
+                       if (old_process)
+                               swapcontext(&old_process->context, &CurrentProcess->context);
+                       else
+                               setcontext(&CurrentProcess->context);
+
+                       // not reached
+               }
+               TRACEMSG("keeping %p:%s", CurrentProcess, CurrentProcess->monitor.name);
+       }
+#endif // CONFIG_KERN_PREEMPT
+}
+
+void irq_register(int irq, void (*callback)(void))
+{
+       irq_handlers[irq] = callback;
+}
+
+void irq_init(void)
+{
+       struct sigaction act;
+       act.sa_handler = irq_entry;
+       sigemptyset(&act.sa_mask);
+       //sigaddset(&act.sa_mask, irq);
+       act.sa_flags = SA_RESTART; // | SA_SIGINFO;
+
+       sigaction(SIGUSR1, &act, NULL);
+       #if !(ARCH & ARCH_QT)
+               sigaction(SIGALRM, &act, NULL);
+       #endif
+
+       MOD_INIT(irq);
+}
diff --git a/bertos/kern/irq.h b/bertos/kern/irq.h
new file mode 100644 (file)
index 0000000..e170ed0
--- /dev/null
@@ -0,0 +1,44 @@
+/**
+ * \file
+ * <!--
+ * This file is part of BeRTOS.
+ *
+ * Bertos is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * As a special exception, you may use this file as part of a free software
+ * library without restriction.  Specifically, if other files instantiate
+ * templates or use macros or inline functions from this file, or you compile
+ * this file and link it with other files to produce an executable, this
+ * file does not by itself cause the resulting executable to be covered by
+ * the GNU General Public License.  This exception does not however
+ * invalidate any other reasons why the executable file might be covered by
+ * the GNU General Public License.
+ *
+ * Copyright 2008 Bernie Innocenti <bernie@codewiz.org>
+ * -->
+ *
+ * \brief Process scheduler (public interface).
+ *
+ * \version $Id: proc.h 1646 2008-08-17 13:49:48Z bernie $
+ * \author Bernie Innocenti <bernie@codewiz.org>
+ */
+#ifndef KERN_IRQ_H
+#define KERN_IRQ_H
+
+void irq_entry(int irq);
+void irq_register(int irq, void (*handler)(void));
+void irq_init();
+
+#endif // KERN_IRQ_H
index 339dc09fe089e8e00e7e38b5e77c04a412f22016..9af6c0d1b402a973cc144532c15c2f1ae5d15d2f 100644 (file)
 #include "proc_p.h"
 #include "proc.h"
 
+#include <kern/irq.h>
 #include <cpu/frame.h> // CPU_IDLE
 #include <drv/timer.h>
+#include <cfg/module.h>
 
 
 /*
@@ -82,8 +84,6 @@ void proc_permit(void)
        --CurrentProcess->forbid_cnt;
 }
 
-static void (*irq_handlers[100])(void); // FIXME
-
 
 void proc_preempt(void)
 {
@@ -156,63 +156,14 @@ void proc_entry(void (*user_entry)(void))
        proc_exit();
 }
 
-/* signal handler */
-void irq_entry(int signum)
-{
-       Process * const old_process = CurrentProcess;
-
-       irq_handlers[signum]();
-
-       if (!CurrentProcess)
-       {
-               TRACEMSG("no runnable processes!");
-               IRQ_ENABLE;
-               pause();
-       }
-       else
-       {
-               if (old_process != CurrentProcess)
-               {
-                       TRACEMSG("switching from %p:%s to %p:%s",
-                               old_process, old_process ? old_process->monitor.name : "-",
-                               CurrentProcess, CurrentProcess->monitor.name);
-
-                       if (old_process)
-                               swapcontext(&old_process->context, &CurrentProcess->context);
-                       else
-                               setcontext(&CurrentProcess->context);
-
-                       /* not reached */
-               }
-               TRACEMSG("keeping %p:%s", CurrentProcess, CurrentProcess->monitor.name);
-       }
-}
-
-void irq_register(int irq, void (*callback)(void))
-{
-       irq_handlers[irq] = callback;
-}
-
-void irq_init(void)
-{
-       struct sigaction act;
-       act.sa_handler = irq_entry;
-       sigemptyset(&act.sa_mask);
-       //sigaddset(&act.sa_mask, irq);
-       act.sa_flags = SA_RESTART; /* | SA_SIGINFO; */
-
-       sigaction(SIGUSR1, &act, NULL);
-       #if !(ARCH & ARCH_QT)
-               sigaction(SIGALRM, &act, NULL);
-       #endif
-}
-
 void preempt_init(void)
 {
-       irq_init(); // FIXME: move before
+       MOD_CHECK(irq);
+       MOD_CHECK(timer);
+
        irq_register(SIGUSR1, proc_preempt);
 
-       timer_setSoftInt(&preempt_timer, proc_preempt_timer, NULL);
+       timer_setSoftint(&preempt_timer, proc_preempt_timer, NULL);
        timer_setDelay(&preempt_timer, CONFIG_KERN_QUANTUM);
        timer_add(&preempt_timer);
 }
index 8692c170840fba17fad60965366d9556d7136715..d4f980af12a244547043a3bf49bcfb9b9facbe19 100644 (file)
@@ -135,7 +135,6 @@ void proc_init(void)
 struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(void), iptr_t data, size_t stack_size, cpustack_t *stack_base)
 {
        Process *proc;
-       size_t i;
        const size_t PROC_SIZE_WORDS = ROUND2(sizeof(Process), sizeof(cpustack_t)) / sizeof(cpustack_t);
 #if CONFIG_KERN_HEAP
        bool free_stack = false;
@@ -200,31 +199,35 @@ struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(voi
        #endif
 #endif
 
-#if CONFIG_KERN_PREEMPT
-       // FIXME: proc_exit
-       getcontext(&proc->context);
-       proc->context.uc_stack.ss_sp = stack_base;
-       proc->context.uc_stack.ss_size = stack_size;
-       proc->context.uc_link = NULL;
-       makecontext(&proc->context, (void (*)(void))proc_entry, 1, entry);
-
-#else // !CONFIG_KERN_PREEMPT
-       /* Initialize process stack frame */
-       CPU_PUSH_CALL_FRAME(proc->stack, proc_exit);
-       CPU_PUSH_CALL_FRAME(proc->stack, entry);
-
-       /* Push a clean set of CPU registers for asm_switch_context() */
-       for (i = 0; i < CPU_SAVED_REGS_CNT; i++)
-               CPU_PUSH_WORD(proc->stack, CPU_REG_INIT_VALUE(i));
-
-       /* Add to ready list */
-       ATOMIC(SCHED_ENQUEUE(proc));
-       ATOMIC(LIST_ASSERT_VALID(&ProcReadyList));
-#endif // CONFIG_KERN_PREEMPT
+       #if CONFIG_KERN_PREEMPT
+               // FIXME: proc_exit
+               getcontext(&proc->context);
+               proc->context.uc_stack.ss_sp = stack_base;
+               proc->context.uc_stack.ss_size = stack_size;
+               proc->context.uc_link = NULL;
+               makecontext(&proc->context, (void (*)(void))proc_entry, 1, entry);
 
-#if CONFIG_KERN_MONITOR
-       monitor_add(proc, name);
-#endif
+       #else // !CONFIG_KERN_PREEMPT
+       {
+               size_t i;
+
+               /* Initialize process stack frame */
+               CPU_PUSH_CALL_FRAME(proc->stack, proc_exit);
+               CPU_PUSH_CALL_FRAME(proc->stack, entry);
+
+               /* Push a clean set of CPU registers for asm_switch_context() */
+               for (i = 0; i < CPU_SAVED_REGS_CNT; i++)
+                       CPU_PUSH_WORD(proc->stack, CPU_REG_INIT_VALUE(i));
+
+               /* Add to ready list */
+               ATOMIC(SCHED_ENQUEUE(proc));
+               ATOMIC(LIST_ASSERT_VALID(&ProcReadyList));
+       }
+       #endif // CONFIG_KERN_PREEMPT
+
+       #if CONFIG_KERN_MONITOR
+               monitor_add(proc, name);
+       #endif
 
        return proc;
 }
index 00ad4a094b8fd74404aeedfbc13c8f9f0512ae60..482fb7a39708f3a1193f1e0b2689322dded7bbbe 100644 (file)
  * 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 Bernie Innocenti <bernie@codewiz.org>
- *
+ * Copyright 2001, 2004 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 1999, 2000, 2001, 2008 Bernie Innocenti <bernie@codewiz.org>
  * -->
  *
  * \brief Process scheduler (public interface).
  *
  * \version $Id$
- *
  * \author Bernie Innocenti <bernie@codewiz.org>
  */
 #ifndef KERN_PROC_H
@@ -142,5 +140,5 @@ void proc_rename(struct Process *proc, const char* name);
                #error No cpustack_t size supported!
        #endif
 #endif
-#endif /* KERN_PROC_H */
 
+#endif /* KERN_PROC_H */