mirrors/Astral
1
0
Fork 0
mirror of https://github.com/Mathewnd/Astral.git synced 2026-11-01 23:18:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

scheduler: fix threadexit interrupt disable

Browse Source
This commit is contained in:
Matheus
2023-07-03 22:48:04 -03:00
parent 00cce59b1a
commit 1621bb1a9b
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
kernel-src/sys/scheduler.c
+3 -1
View File
@@ -196,7 +196,6 @@ __attribute__((noreturn)) void sched_stopcurrentthread() {
} }
__attribute__((noreturn)) void sched_threadexit() { __attribute__((noreturn)) void sched_threadexit() {
interrupt_set(false);
thread_t *thread = _cpu()->thread; thread_t *thread = _cpu()->thread;
proc_t *proc = thread->proc; proc_t *proc = thread->proc;
@@ -205,10 +204,12 @@ __attribute__((noreturn)) void sched_threadexit() {
if (proc) { if (proc) {
__atomic_fetch_sub(&proc->runningthreadcount, 1, __ATOMIC_SEQ_CST); __atomic_fetch_sub(&proc->runningthreadcount, 1, __ATOMIC_SEQ_CST);
__assert(oldctx != &vmm_kernelctx);
if (proc->runningthreadcount == 0) if (proc->runningthreadcount == 0)
vmm_destroycontext(oldctx); vmm_destroycontext(oldctx);
} }
interrupt_set(false);
_cpu()->thread = NULL; _cpu()->thread = NULL;
// because a thread deallocating its own data is a nightmare, thread deallocation and such will be left to whoever frees the proc it's tied to // because a thread deallocating its own data is a nightmare, thread deallocation and such will be left to whoever frees the proc it's tied to
@@ -229,6 +230,7 @@ static void yield(context_t *context) {
if (next || sleeping) { if (next || sleeping) {
ARCH_CONTEXT_THREADSAVE(thread, context); ARCH_CONTEXT_THREADSAVE(thread, context);
if (sleeping) if (sleeping)
spinlock_release(&thread->sleeplock); spinlock_release(&thread->sleeplock);
else else