preempt: Move idle process to its own source file

git-svn-id: https://src.develer.com/svnoss/bertos/trunk@1672 38d2e660-2303-0410-9eaa-f027e97ec537

diff --git a/app/demo/demo.mk b/app/demo/demo.mk
index 51b71bd03237a7954254365c99c6a67a71640020..920f2a6a7074ea97dfdab7777a8968010e31f2e0 100644 (file)
--- a/app/demo/demo.mk
+++ b/app/demo/demo.mk
@@ -1,10 +1,10 @@
 #
-# $Id: demo.mk 18234 2007-10-08 13:39:48Z rasky $
 # Copyright 2003, 2004, 2005, 2006 Develer S.r.l. (http://www.develer.com/)
-# All rights reserved.
+# Copyright 2008 Bernie Innocenti <bernie@codewiz.org>
 #
 # Makefile fragment for DevLib demo application.
 #
+# Version: $Id: demo.mk 18234 2007-10-08 13:39:48Z rasky $
 # Author: Bernie Innocenti <bernie@codewiz.org>
 #
 
@@ -52,6 +52,7 @@ demo_CSRC = \
        bertos/mware/observer.c \
        bertos/mware/resource.c \
        bertos/mware/sprintf.c \
+       bertos/kern/idle.c \
        bertos/kern/irq.c \
        bertos/kern/preempt.c \
        bertos/kern/proc.c \

diff --git a/bertos/kern/idle.c b/bertos/kern/idle.c
new file mode 100644 (file)
index 0000000..ede234c
--- /dev/null
+++ b/bertos/kern/idle.c
@@ -0,0 +1,70 @@
+/**
+ * \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 Idle loop for preemptive scheduling
+ *
+ * \version $Id: proc.c 1616 2008-08-10 19:41:26Z bernie $
+ * \author Bernie Innocenti <bernie@codewiz.org>
+ */
+
+#include "proc.h"
+
+#include <cfg/module.h>
+
+
+static cpustack_t idle_stack[CONFIG_PROC_DEFSTACKSIZE / sizeof(cpustack_t)];
+
+/**
+ * The idle process
+ *
+ * This process never dies and never sleeps.  It's also quite lazy, apathic
+ * and sometimes even a little antisocial.
+ *
+ * Having an idle process costs us some stack space, but simplifies the
+ * interrupt-driven preemption logic because there is always a user
+ * context to which we can return.
+ *
+ * The idle process is not required for cooperative task switching.
+ */
+static NORETURN void idle(void)
+{
+       for (;;)
+       {
+               TRACE;
+               //monitor_report();
+               proc_yield(); // FIXME: CPU_IDLE
+       }
+}
+
+void idle_init(void)
+{
+       proc_new(idle, NULL, sizeof(idle_stack), idle_stack);
+}

diff --git a/bertos/kern/preempt.c b/bertos/kern/preempt.c
index 756a38fe0b89d1073b0c4585f8b702050877901a..cde336510ee0e36cf6644f9bc5f92e5f0e74dc98 100644 (file)
--- a/bertos/kern/preempt.c
+++ b/bertos/kern/preempt.c
@@ -51,6 +51,10 @@ int preempt_forbid_cnt;
 Timer preempt_timer;
 
 
+// fwd decl from idle.c
+void idle_init(void);
+
+
 void proc_preempt(void)
 {
        IRQ_DISABLE;
@@ -118,30 +122,6 @@ void proc_entry(void (*user_entry)(void))
        proc_exit();
 }
 
-
-static cpustack_t idle_stack[CONFIG_PROC_DEFSTACKSIZE / sizeof(cpustack_t)];
-
-// FIXME: move this to kern/idle.c
-/**
- * The idle process
- *
- * This process never dies and never sleeps.  It's also quite lazy, apathic
- * and a bit antisocial.
- *
- * Having an idle process costs some stack space, but simplifies the
- * interrupt-driven preemption logic because there is always a user
- * context to which we can return.
- */
-static NORETURN void idle(void)
-{
-       for (;;)
-       {
-               TRACE;
-               //monitor_report();
-               proc_yield(); // FIXME: CPU_IDLE
-       }
-}
-
 void preempt_init(void)
 {
        MOD_CHECK(irq);
@@ -153,5 +133,5 @@ void preempt_init(void)
        timer_setDelay(&preempt_timer, CONFIG_KERN_QUANTUM);
        timer_add(&preempt_timer);
 
-       proc_new(idle, NULL, sizeof(idle_stack), idle_stack);
+       idle_init();
 }