Changes in libcfa/src/concurrency/kernel.cfa [0190480:58d64a4]

File:

• libcfa/src/concurrency/kernel.cfa (modified) (6 diffs)

libcfa/src/concurrency/kernel.cfa

@@ -237 +237 @@
         $coroutine * proc_cor = get_coroutine(this->runner);
 
-        // Update global state
-        kernelTLS.this_thread = thrd_dst;
-
         // set state of processor coroutine to inactive
         verify(proc_cor->state == Active);
@@ -253 +250 @@
                         thrd_dst->unpark_stale = true;
                 )
+                // Update global state
+                kernelTLS.this_thread = thrd_dst;
 
                 /* paranoid */ verify( ! kernelTLS.preemption_state.enabled );
@@ -259 +258 @@
                 /* paranoid */ verifyf( ((uintptr_t)thrd_dst->context.SP) > ((uintptr_t)__get_stack(thrd_dst->curr_cor)->limit) || thrd_dst->curr_cor == proc_cor, "ERROR : Destination $thread %p has been corrupted.\n StackPointer too large.\n", thrd_dst ); // add escape condition if we are setting up the processor
 
+
                 // set context switch to the thread that the processor is executing
                 verify( thrd_dst->context.SP );
@@ -269 +269 @@
                 /* paranoid */ verify( ! kernelTLS.preemption_state.enabled );
 
+                // Reset global state
+                kernelTLS.this_thread = 0p;
 
                 // We just finished running a thread, there are a few things that could have happened.
@@ -313 +315 @@
         // Just before returning to the processor, set the processor coroutine to active
         proc_cor->state = Active;
-        kernelTLS.this_thread = 0p;
 
         /* paranoid */ verify( ! kernelTLS.preemption_state.enabled );
@@ -521 +522 @@
         disable_interrupts();
                 /* paranoid */ verify( ! kernelTLS.preemption_state.enabled );
-                bool ret = post( this->idle );
+                post( this->idle );
         enable_interrupts( __cfaabi_dbg_ctx );
 }