X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fkern%2Fproc.c;h=c422e729c6912963f51bfcd89f75a7cee269e290;hb=ae8a609173e4490fd03875f96e388038053b9288;hp=afe9e34a92e5430a9832d2d76a0d2ce2907c15d1;hpb=0315f64ec2ee2c5eb10634360fb8874c52dbbabf;p=bertos.git diff --git a/bertos/kern/proc.c b/bertos/kern/proc.c index afe9e34a..c422e729 100644 --- a/bertos/kern/proc.c +++ b/bertos/kern/proc.c @@ -30,10 +30,8 @@ * 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 * \author Stefano Fedrigo */ @@ -41,20 +39,28 @@ #include "proc_p.h" #include "proc.h" -#include "cfg/cfg_arch.h" /* ARCH_EMUL */ -#include "cfg/cfg_kern.h" +#include "cfg/cfg_proc.h" +#define LOG_LEVEL KERN_LOG_LEVEL +#define LOG_FORMAT KERN_LOG_FORMAT +#include + +#include "cfg/cfg_monitor.h" +#include // ROUND_UP2 #include +#include // CONFIG_DEPEND() #include #include #include #include +#if CONFIG_KERN_HEAP + #include +#endif + #include /* memset() */ -#if CONFIG_KERN_PREEMPT -#include "preempt.h" -#endif +#define PROC_SIZE_WORDS (ROUND_UP2(sizeof(Process), sizeof(cpu_stack_t)) / sizeof(cpu_stack_t)) /* * The scheduer tracks ready processes by enqueuing them in the @@ -62,139 +68,225 @@ * * \note Access to the list must occur while interrupts are disabled. */ -REGISTER List ProcReadyList; +REGISTER List proc_ready_list; /* * Holds a pointer to the TCB of the currently running process. * * \note User applications should use proc_current() to retrieve this value. */ -REGISTER Process *CurrentProcess; +REGISTER Process *current_process; + +/** The main process (the one that executes main()). */ +static struct Process main_process; + +#if CONFIG_KERN_HEAP + +/** + * Local heap dedicated to allocate the memory used by the processes. + */ +static HEAP_DEFINE_BUF(heap_buf, CONFIG_KERN_HEAP_SIZE); +static Heap proc_heap; -#if (ARCH & ARCH_EMUL) /* - * In hosted environments, we must emulate the stack on the real process stack. + * Keep track of zombie processes (processes that are exiting and need to + * release some resources). * - * Access to this list must be protected by PROC_ATOMIC(). + * \note Access to the list must occur while kernel preemption is disabled. */ -extern List StackFreeList; -#endif - -/** The main process (the one that executes main()). */ -struct Process MainProcess; +static List zombie_list; +#endif /* CONFIG_KERN_HEAP */ -static void proc_init_struct(Process *proc) +static void proc_initStruct(Process *proc) { /* Avoid warning for unused argument. */ (void)proc; #if CONFIG_KERN_SIGNALS proc->sig_recv = 0; -#endif - -#if CONFIG_KERN_PREEMPTIVE - proc->forbid_cnt = 0; + proc->sig_wait = 0; #endif #if CONFIG_KERN_HEAP proc->flags = 0; #endif + +#if CONFIG_KERN_PRI + proc->link.pri = 0; +#endif } MOD_DEFINE(proc); void proc_init(void) { - LIST_INIT(&ProcReadyList); + LIST_INIT(&proc_ready_list); +#if CONFIG_KERN_HEAP + LIST_INIT(&zombie_list); + heap_init(&proc_heap, heap_buf, sizeof(heap_buf)); +#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 * pointer because it will be written the first time we switch to another process. */ - proc_init_struct(&MainProcess); - CurrentProcess = &MainProcess; + proc_initStruct(&main_process); + current_process = &main_process; #if CONFIG_KERN_MONITOR monitor_init(); - monitor_add(CurrentProcess, "main"); + monitor_add(current_process, "main"); #endif + proc_schedInit(); + + MOD_INIT(proc); +} + -#if CONFIG_KERN_PREEMPTIVE - preempt_init(); +#if CONFIG_KERN_HEAP + +/** + * Free all the resources of all zombie processes previously added to the zombie + * list. + */ +static void proc_freeZombies(void) +{ + Process *proc; + + while (1) + { + PROC_ATOMIC(proc = (Process *)list_remHead(&zombie_list)); + if (proc == NULL) + return; + + if (proc->flags & PF_FREESTACK) + { + PROC_ATOMIC(heap_freemem(&proc_heap, proc->stack_base, + proc->stack_size + PROC_SIZE_WORDS * sizeof(cpu_stack_t))); + } + } +} + +/** + * Enqueue a process in the zombie list. + */ +static void proc_addZombie(Process *proc) +{ + Node *node; +#if CONFIG_KERN_PREEMPT + ASSERT(!proc_preemptAllowed()); #endif - MOD_INIT(proc); +#if CONFIG_KERN_PRI + node = &(proc)->link.link; +#else + node = &(proc)->link; +#endif + LIST_ASSERT_VALID(&zombie_list); + ADDTAIL(&zombie_list, node); } +#endif /* CONFIG_KERN_HEAP */ /** * Create a new process, starting at the provided entry point. * + * + * \note The function + * \code + * proc_new(entry, data, stacksize, stack) + * \endcode + * is a more convenient way to create a process, as you don't have to specify + * the name. + * * \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; - size_t i; - const size_t PROC_SIZE_WORDS = ROUND2(sizeof(Process), sizeof(cpustack_t)) / sizeof(cpustack_t); + LOG_INFO("name=%s", name); #if CONFIG_KERN_HEAP bool free_stack = false; -#endif - TRACEMSG("name=%s", name); -#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)); + /* + * Free up resources of a zombie process. + * + * We're implementing a kind of lazy garbage collector here for + * efficiency reasons: we can avoid to introduce overhead into another + * kernel task dedicated to free up resources (e.g., idle) and we're + * not introducing any overhead into the scheduler after a context + * switch (that would be *very* bad, because the scheduler runs with + * IRQ disabled). + * + * In this way we are able to release the memory of the zombie tasks + * without disabling IRQs and without introducing any significant + * overhead in any other kernel task. + */ + proc_freeZombies(); - stack_size = CONFIG_PROC_DEFSTACKSIZE; -#elif CONFIG_KERN_HEAP /* Did the caller provide a stack for us? */ if (!stack_base) { /* Did the caller specify the desired stack size? */ if (!stack_size) - stack_size = CONFIG_PROC_DEFSTACKSIZE + sizeof(Process); + stack_size = KERN_MINSTACKSIZE; /* Allocate stack dinamically */ - if (!(stack_base = heap_alloc(stack_size))) + PROC_ATOMIC(stack_base = + (cpu_stack_t *)heap_allocmem(&proc_heap, stack_size)); + if (stack_base == NULL) return NULL; free_stack = true; } -#else + +#else // CONFIG_KERN_HEAP + /* Stack must have been provided by the user */ ASSERT_VALID_PTR(stack_base); ASSERT(stack_size); -#endif + +#endif // 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 */ if (CPU_STACK_GROWS_UPWARD) { - proc = (Process*)stack_base; + proc = (Process *)stack_base; proc->stack = stack_base + PROC_SIZE_WORDS; + // On some architecture stack should be aligned, so we do it. + proc->stack = (cpu_stack_t *)((uintptr_t)proc->stack + (sizeof(cpu_aligned_stack_t) - ((uintptr_t)proc->stack % sizeof(cpu_aligned_stack_t)))); if (CPU_SP_ON_EMPTY_SLOT) proc->stack++; } 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); + // On some architecture stack should be aligned, so we do it. + proc->stack = (cpu_stack_t *)((uintptr_t)proc - ((uintptr_t)proc % sizeof(cpu_aligned_stack_t))); if (CPU_SP_ON_EMPTY_SLOT) proc->stack--; } + /* Ensure stack is aligned */ + ASSERT((uintptr_t)proc->stack % sizeof(cpu_aligned_stack_t) == 0); - proc_init_struct(proc); + stack_size -= PROC_SIZE_WORDS * sizeof(cpu_stack_t); + proc_initStruct(proc); proc->user_data = data; -#if CONFIG_KERN_HEAP | CONFIG_KERN_MONITOR | (ARCH & ARCH_EMUL) +#if CONFIG_KERN_HEAP | CONFIG_KERN_MONITOR proc->stack_base = stack_base; proc->stack_size = stack_size; #if CONFIG_KERN_HEAP @@ -202,37 +294,41 @@ struct Process *proc_new_with_name(UNUSED(const char *, name), void (*entry)(voi proc->flags |= PF_FREESTACK; #endif #endif + proc->user_entry = entry; + CPU_CREATE_NEW_STACK(proc->stack); -#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, entry, 0); - -#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)); +#if CONFIG_KERN_MONITOR + monitor_add(proc, name); +#endif /* Add to ready list */ ATOMIC(SCHED_ENQUEUE(proc)); - ATOMIC(LIST_ASSERT_VALID(&ProcReadyList)); -#endif // CONFIG_KERN_PREEMPT + return proc; +} + +/** + * Return the name of the specified process. + * + * NULL is a legal argument and will return the name "". + */ +const char *proc_name(struct Process *proc) +{ #if CONFIG_KERN_MONITOR - monitor_add(proc, name); + return proc ? proc->monitor.name : ""; +#else + (void)proc; + return "---"; #endif +} - return proc; +/// Return the name of the currently running process +const char *proc_currentName(void) +{ + return proc_name(proc_current()); } -/** Rename a process */ +/// Rename a process void proc_rename(struct Process *proc, const char *name) { #if CONFIG_KERN_MONITOR @@ -243,61 +339,94 @@ void proc_rename(struct Process *proc, const char *name) } +#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 != current_process) + ATOMIC(sched_reenqueue(proc)); +} +#endif // CONFIG_KERN_PRI + +INLINE void proc_run(void) +{ + void (*entry)(void) = current_process->user_entry; + + LOG_INFO("New process starting at %p", entry); + entry(); +} + +/** + * Entry point for all the processes. + */ +void proc_entry(void) +{ + /* + * Return from a context switch assumes interrupts are disabled, so + * we need to explicitly re-enable them as soon as possible. + */ + IRQ_ENABLE; + /* Call the actual process's entry point */ + proc_run(); + proc_exit(); +} + /** * Terminate the current process */ void proc_exit(void) { - TRACE; + LOG_INFO("%p:%s", current_process, proc_currentName()); #if CONFIG_KERN_MONITOR - monitor_remove(CurrentProcess); + monitor_remove(current_process); #endif + proc_forbid(); #if CONFIG_KERN_HEAP /* - * 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 - * after a context switch. + * Set the task as zombie, its resources will be freed in proc_new() in + * a lazy way, when another process will be created. */ - if (CurrentProcess->flags & PF_FREESTACK) - heap_free(CurrentProcess->stack_base, CurrentProcess->stack_size); - heap_free(CurrentProcess); + proc_addZombie(current_process); #endif + current_process = NULL; + proc_permit(); -#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))))); - - /* - * 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 */ + proc_switch(); - CurrentProcess = NULL; - proc_schedule(); - /* not reached */ + /* never reached */ + ASSERT(0); } -/** - * Get the pointer to the current process - */ -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) +iptr_t proc_currentUserData(void) { - return CurrentProcess->user_data; + return current_process->user_data; }