X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fkern%2Fproc.c;h=2cb6e2a0aa3a31d3ee9983059cbefa52fe3bbf67;hb=e62ca0b357f09804d7d894949df44224c9d74bb7;hp=0e3edbea37e368bfac54003ee9667d1cc0c7f042;hpb=0cd2e04de03f9c793a9dd9a564124a5b699455ba;p=bertos.git diff --git a/bertos/kern/proc.c b/bertos/kern/proc.c index 0e3edbea..2cb6e2a0 100644 --- a/bertos/kern/proc.c +++ b/bertos/kern/proc.c @@ -30,8 +30,7 @@ * Copyright 1999, 2000, 2001, 2008 Bernie Innocenti * --> * - * \brief Simple realtime multitasking scheduler. - * Context switching is only done cooperatively. + * \brief Simple cooperative multitasking scheduler. * * \version $Id$ * \author Bernie Innocenti @@ -41,17 +40,28 @@ #include "proc_p.h" #include "proc.h" -#include "cfg/cfg_arch.h" /* ARCH_EMUL */ +#include "cfg/cfg_arch.h" // ARCH_EMUL #include "cfg/cfg_kern.h" +#include // ROUND_UP2 #include +#include // CONFIG_DEPEND() #include #include #include #include +#if CONFIG_KERN_HEAP + #include +#endif + #include /* memset() */ +// Check config dependencies +CONFIG_DEPEND(CONFIG_KERN_SIGNALS, CONFIG_KERN_SCHED); +CONFIG_DEPEND(CONFIG_KERN_SEMAPHORES, CONFIG_KERN_SIGNALS); +CONFIG_DEPEND(CONFIG_KERN_MONITOR, CONFIG_KERN_SCHED); + /* * The scheduer tracks ready processes by enqueuing them in the @@ -70,11 +80,16 @@ REGISTER Process *CurrentProcess; #if (ARCH & ARCH_EMUL) /* - * In hosted environments, we must emulate the stack on the real process stack. + * In some hosted environments, we must emulate the stack on the real + * process stack to satisfy consistency checks in system libraries and + * because some ABIs place trampolines on the stack. * * Access to this list must be protected by PROC_ATOMIC(). */ -extern List StackFreeList; +List StackFreeList; + +#define NPROC 8 +cpu_stack_t proc_stacks[NPROC][(64 * 1024) / sizeof(cpu_stack_t)]; #endif /** The main process (the one that executes main()). */ @@ -93,6 +108,10 @@ static void proc_init_struct(Process *proc) #if CONFIG_KERN_HEAP proc->flags = 0; #endif + +#if CONFIG_KERN_PRI + proc->link.pri = 0; +#endif } MOD_DEFINE(proc); @@ -101,6 +120,12 @@ void proc_init(void) { LIST_INIT(&ProcReadyList); +#if ARCH & ARCH_EMUL + LIST_INIT(&StackFreeList); + for (int i = 0; i < NPROC; i++) + ADDTAIL(&StackFreeList, (Node *)proc_stacks[i]); +#endif + /* * We "promote" the current context into a real process. The only thing we have * to do is create a PCB and make it current. We don't need to setup the stack @@ -121,17 +146,16 @@ void proc_init(void) MOD_INIT(proc); } - /** * 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 stack_size, cpustack_t *stack_base) +struct Process *proc_new_with_name(UNUSED_ARG(const char *, name), void (*entry)(void), iptr_t data, size_t stack_size, cpu_stack_t *stack_base) { Process *proc; - const size_t PROC_SIZE_WORDS = ROUND2(sizeof(Process), sizeof(cpustack_t)) / sizeof(cpustack_t); + const size_t PROC_SIZE_WORDS = ROUND_UP2(sizeof(Process), sizeof(cpu_stack_t)) / sizeof(cpu_stack_t); #if CONFIG_KERN_HEAP bool free_stack = false; #endif @@ -139,7 +163,7 @@ struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(voi #if (ARCH & ARCH_EMUL) /* Ignore stack provided by caller and use the large enough default instead. */ - PROC_ATOMIC(stack_base = (cpustack_t *)list_remHead(&StackFreeList)); + PROC_ATOMIC(stack_base = (cpu_stack_t *)list_remHead(&StackFreeList)); stack_size = CONFIG_KERN_MINSTACKSIZE; #elif CONFIG_KERN_HEAP @@ -166,8 +190,13 @@ struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(voi #endif // !ARCH_EMUL && !CONFIG_KERN_HEAP #if CONFIG_KERN_MONITOR - /* Fill-in the stack with a special marker to help debugging */ - memset(stack_base, CONFIG_KERN_STACKFILLCODE, stack_size); + /* + * Fill-in the stack with a special marker to help debugging. + * On 64bit platforms, CONFIG_KERN_STACKFILLCODE is larger + * than an int, so the (int) cast is required to silence the + * warning for truncating its size. + */ + memset(stack_base, (int)CONFIG_KERN_STACKFILLCODE, stack_size); #endif /* Initialize the process control block */ @@ -180,8 +209,8 @@ struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(voi } else { - proc = (Process *)(stack_base + stack_size / sizeof(cpustack_t) - PROC_SIZE_WORDS); - proc->stack = (cpustack_t *)proc; + proc = (Process *)(stack_base + stack_size / sizeof(cpu_stack_t) - PROC_SIZE_WORDS); + proc->stack = (cpu_stack_t *)proc; if (CPU_SP_ON_EMPTY_SLOT) proc->stack--; } @@ -261,6 +290,43 @@ void proc_rename(struct Process *proc, const char *name) #endif } + +#if CONFIG_KERN_PRI +/** + * Change the scheduling priority of a process. + * + * Process piorities are signed ints, whereas a larger integer value means + * higher scheduling priority. The default priority for new processes is 0. + * The idle process runs with the lowest possible priority: INT_MIN. + * + * A process with a higher priority always preempts lower priority processes. + * Processes of equal priority share the CPU time according to a simple + * round-robin policy. + * + * As a general rule to maximize responsiveness, compute-bound processes + * should be assigned negative priorities and tight, interactive processes + * should be assigned positive priorities. + * + * To avoid interfering with system background activities such as input + * processing, application processes should remain within the range -10 + * and +10. + */ +void proc_setPri(struct Process *proc, int pri) +{ + if (proc->link.pri == pri) + return; + + proc->link.pri = pri; + + if (proc != CurrentProcess) + { + //proc_forbid(); + //TODO: re-enqueue process + //pric_permit(); + } +} +#endif // CONFIG_KERN_PRI + /** * Terminate the current process */ @@ -286,10 +352,8 @@ void proc_exit(void) #endif #if (ARCH & ARCH_EMUL) -#warning This is wrong /* Reinsert process stack in free list */ - PROC_ATOMIC(ADDHEAD(&StackFreeList, (Node *)(CurrentProcess->stack - - (CONFIG_PROC_DEFSTACKSIZE / sizeof(cpustack_t))))); + PROC_ATOMIC(ADDHEAD(&StackFreeList, (Node *)CurrentProcess->stack_base)); /* * NOTE: At this point the first two words of what used @@ -299,7 +363,7 @@ void proc_exit(void) #endif /* ARCH_EMUL */ CurrentProcess = NULL; - proc_schedule(); + proc_switch(); /* not reached */ }