X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=kern%2Fproc.c;h=15817ef79803ea26a9ded4679b14a9bdf6e711fa;hb=6cc971430fda3760940c8f92bb484a6145bbad8d;hp=2301cee05c862bc5fe86c7722363c50e39db5d8a;hpb=5b25ae54d86d894e5d01adeef10938df1bb10e16;p=bertos.git diff --git a/kern/proc.c b/kern/proc.c index 2301cee0..15817ef7 100755 --- a/kern/proc.c +++ b/kern/proc.c @@ -1,9 +1,9 @@ -/*! +/** * \file * * * \brief Simple realtime multitasking scheduler. @@ -17,6 +17,36 @@ /*#* *#* $Log$ + *#* Revision 1.32 2006/09/20 14:19:23 marco + *#* Restored test. + *#* + *#* Revision 1.31 2006/07/19 12:56:27 bernie + *#* Convert to new Doxygen style. + *#* + *#* Revision 1.30 2006/03/27 04:49:23 bernie + *#* CPU_IDLE(): Fix for new emulator. + *#* + *#* Revision 1.29 2006/02/24 01:17:05 bernie + *#* Update for new emulator. + *#* + *#* Revision 1.28 2006/02/21 16:06:55 bernie + *#* Cleanup/update process scheduling. + *#* + *#* Revision 1.27 2005/11/04 16:20:02 bernie + *#* Fix reference to README.devlib in header. + *#* + *#* Revision 1.26 2005/04/11 19:10:28 bernie + *#* Include top-level headers from cfg/ subdir. + *#* + *#* Revision 1.25 2005/03/15 00:20:54 bernie + *#* proc_schedule(): New sanity check. + *#* + *#* Revision 1.24 2005/01/08 09:20:54 bernie + *#* Remove unused variable. + *#* + *#* Revision 1.23 2004/12/13 12:07:06 bernie + *#* DISABLE_IRQSAVE/ENABLE_IRQRESTORE: Convert to IRQ_SAVE_DISABLE/IRQ_RESTORE. + *#* *#* Revision 1.22 2004/12/13 11:51:08 bernie *#* DISABLE_INTS/ENABLE_INTS: Convert to IRQ_DISABLE/IRQ_ENABLE. *#* @@ -85,16 +115,16 @@ #include "proc_p.h" #include "proc.h" -#include "cpu.h" -#include "event.h" -#include "hw.h" -#include -#include /* ARCH_EMUL */ -#include /* ABS() */ +//#include "hw.h" +#include +#include +#include +#include /* ARCH_EMUL */ +#include /* ABS() */ #include /* memset() */ -/*! +/** * CPU dependent context switching routines. * * \note This function *MUST* preserve also the status of the interrupts. @@ -127,7 +157,7 @@ uint16_t Quantum; extern List StackFreeList; #endif -/*! The main process (the one that executes main()). */ +/** The main process (the one that executes main()). */ struct Process MainProcess; @@ -170,16 +200,15 @@ void proc_init(void) } -/*! +/** * Create a new process, starting at the provided entry point. * * \return Process structure of new created process * if successful, NULL otherwise. */ -struct Process *proc_new_with_name(UNUSED(const char*, name), void (*entry)(void), iptr_t data, size_t stacksize, cpustack_t *stack_base) +struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(void), iptr_t data, size_t stacksize, cpustack_t *stack_base) { Process *proc; - cpuflags_t flags; size_t i; size_t proc_size_words = ROUND2(sizeof(Process), sizeof(cpustack_t)) / sizeof(cpustack_t); #if CONFIG_KERN_HEAP @@ -188,9 +217,9 @@ struct Process *proc_new_with_name(UNUSED(const char*, name), void (*entry)(void #if (ARCH & ARCH_EMUL) /* Ignore stack provided by caller and use the large enough default instead. */ - stack_base = (cpustack_t *)StackFreeList.head; - REMOVE((Node *)stack_base); - stacksize = DEF_STACKSIZE; + stack_base = (cpustack_t *)LIST_HEAD(&StackFreeList); + REMOVE(LIST_HEAD(&StackFreeList)); + stacksize = CONFIG_KERN_DEFSTACKSIZE; #elif CONFIG_KERN_HEAP /* Did the caller provide a stack for us? */ if (!stack_base) @@ -251,9 +280,7 @@ struct Process *proc_new_with_name(UNUSED(const char*, name), void (*entry)(void CPU_PUSH_WORD(proc->stack, CPU_REG_INIT_VALUE(i)); /* Add to ready list */ - DISABLE_IRQSAVE(flags); - SCHED_ENQUEUE(proc); - ENABLE_IRQRESTORE(flags); + ATOMIC(SCHED_ENQUEUE(proc)); #if CONFIG_KERN_MONITOR monitor_add(proc, name, stack_base, stacksize); @@ -262,7 +289,7 @@ struct Process *proc_new_with_name(UNUSED(const char*, name), void (*entry)(void return proc; } -/*! Rename a process */ +/** Rename a process */ void proc_rename(struct Process *proc, const char *name) { #if CONFIG_KERN_MONITOR @@ -273,7 +300,7 @@ void proc_rename(struct Process *proc, const char *name) } -/*! +/** * System scheduler: pass CPU control to the next process in * the ready queue. * @@ -293,9 +320,14 @@ void proc_schedule(void) /* Remember old process to save its context later */ old_process = CurrentProcess; +#ifdef IRQ_RUNNING + /* Scheduling in interrupts is a nono. */ + ASSERT(!IRQ_RUNNING()); +#endif + /* Poll on the ready queue for the first ready process */ - DISABLE_IRQSAVE(flags); - while (!(CurrentProcess = (struct Process *)REMHEAD(&ProcReadyList))) + IRQ_SAVE_DISABLE(flags); + while (!(CurrentProcess = (struct Process *)list_remHead(&ProcReadyList))) { /* * Make sure we physically reenable interrupts here, no matter what @@ -308,10 +340,10 @@ void proc_schedule(void) * reason to do this. */ IRQ_ENABLE; - SCHEDULER_IDLE; + CPU_IDLE; IRQ_DISABLE; } - ENABLE_IRQRESTORE(flags); + IRQ_RESTORE(flags); /* * Optimization: don't switch contexts when the active @@ -340,13 +372,18 @@ void proc_schedule(void) } -/*! +/** * Terminate the current process */ void proc_exit(void) { +#if CONFIG_KERN_MONITOR + monitor_remove(CurrentProcess); +#endif + #if CONFIG_KERN_HEAP - /* The following code is BROKEN. + /* + * The following code is BROKEN. * We are freeing our own stack before entering proc_schedule() * BAJO: A correct fix would be to rearrange the scheduler with * an additional parameter which frees the old stack/process @@ -358,28 +395,25 @@ void proc_exit(void) #endif #if (ARCH & ARCH_EMUL) -#error This is wrong +#warning This is wrong /* Reinsert process stack in free list */ ADDHEAD(&StackFreeList, (Node *)(CurrentProcess->stack - - (DEF_STACKSIZE / sizeof(cpustack_t)))); + - (CONFIG_KERN_DEFSTACKSIZE / sizeof(cpustack_t)))); - /* NOTE: At this point the first two words of what used + /* + * NOTE: At this point the first two words of what used * to be our stack contain a list node. From now on, we * rely on the compiler not reading/writing the stack. */ #endif /* ARCH_EMUL */ -#if CONFIG_KERN_MONITOR - monitor_remove(CurrentProcess); -#endif - CurrentProcess = NULL; proc_schedule(); /* not reached */ } -/*! +/** * Co-operative context switch */ void proc_switch(void) @@ -387,15 +421,15 @@ void proc_switch(void) /* Just like proc_schedule, this function must not have auto variables. */ static cpuflags_t flags; - DISABLE_IRQSAVE(flags); + IRQ_SAVE_DISABLE(flags); SCHED_ENQUEUE(CurrentProcess); - ENABLE_IRQRESTORE(flags); + IRQ_RESTORE(flags); proc_schedule(); } -/*! +/** * Get the pointer to the current process */ struct Process *proc_current(void) @@ -403,7 +437,7 @@ struct Process *proc_current(void) return CurrentProcess; } -/*! +/** * Get the pointer to the user data of the current process */ iptr_t proc_current_user_data(void) @@ -414,7 +448,7 @@ iptr_t proc_current_user_data(void) #if CONFIG_KERN_PREEMPTIVE -/*! +/** * Disable preemptive task switching. * * The scheduler maintains a per-process nesting counter. Task switching is @@ -434,7 +468,7 @@ void proc_forbid(void) ++CurrentProcess->forbid_cnt; } -/*! +/** * Re-enable preemptive task switching. * * \sa proc_forbid() @@ -446,62 +480,3 @@ void proc_permit(void) } #endif /* CONFIG_KERN_PREEMPTIVE */ - - -#if 0 /* Simple testcase for the scheduler */ - -#include - -/*! - * Proc scheduling test subthread 1 - */ -static void NORETURN proc_test_thread1(void) -{ - for (;;) - { - kputs(">task 1\n"); - timer_delay(50); - proc_switch(); - } -} - -/*! - * Proc scheduling test subthread 2 - */ -static void NORETURN proc_test_thread2(void) -{ - for (;;) - { - kputs(">task 2\n"); - timer_delay(75); - proc_switch(); - } -} - -static cpustack_t proc_test_stack1[CONFIG_KERN_DEFSTACKSIZE/sizeof(cpustack_t)]; -static cpustack_t proc_test_stack2[CONFIG_KERN_DEFSTACKSIZE/sizeof(cpustack_t)]; - -/*! - * Proc scheduling test - */ -void NORETURN proc_test(void) -{ - proc_new(proc_test_thread1, NULL, sizeof(proc_test_stack1), proc_test_stack1); - proc_new(proc_test_thread2, NULL, sizeof(proc_test_stack2), proc_test_stack2); - kputs("Created tasks\n"); - - kputs("stack1:\n"); - kdump(proc_test_stack1+sizeof(proc_test_stack1)-64, 64); - kputs("stack2:\n"); - kdump(proc_test_stack2+sizeof(proc_test_stack1)-64, 64); - - for (;;) - { - kputs(">main task\n"); - timer_delay(93); - proc_switch(); - } - - ASSERT(false); -} -#endif