Changes in src/libcfa/concurrency/thread.c [de94a60:b10affd]

File:

• src/libcfa/concurrency/thread.c (modified) (7 diffs)

src/libcfa/concurrency/thread.c

@@ -30 +30 @@
 //-----------------------------------------------------------------------------
 // Thread ctors and dtors
-void ?{}(thread_desc & this, const char * const name, cluster & cl, void * storage, size_t storageSize ) with( this ) {
-        self_cor{ name, storage, storageSize };
-        verify(&self_cor);
+
+void ?{}(thread_desc& this) with( this ) {
+        self_cor{};
+        self_cor.name = "Anonymous Coroutine";
         curr_cor = &self_cor;
         self_mon.owner = &this;
         self_mon.recursion = 1;
         self_mon_p = &self_mon;
-        curr_cluster = &cl;
         next = NULL;
-
-        node.next = NULL;
-        node.prev = NULL;
-        doregister(this);
+        __cfaabi_dbg_debug_do(
+                dbg_next = NULL;
+                dbg_prev = NULL;
+                __cfaabi_dbg_thread_register(&this);
+        )
 
         monitors{ &self_mon_p, 1, (fptr_t)0 };
@@ -48 +49 @@
 
 void ^?{}(thread_desc& this) with( this ) {
-        unregister(this);
         ^self_cor{};
 }
@@ -81 +81 @@
         disable_interrupts();
         create_stack(&thrd_c->stack, thrd_c->stack.size);
-        kernelTLS.this_coroutine = thrd_c;
+        TL_SET( this_coroutine, thrd_c );
         CtxStart(&this, CtxInvokeThread);
         assert( thrd_c->last->stack.context );
@@ -91 +91 @@
 
 extern "C" {
-        // KERNEL ONLY
         void __finish_creation(void) {
-                coroutine_desc* thrd_c = kernelTLS.this_coroutine;
+                coroutine_desc* thrd_c = TL_GET( this_coroutine );
                 ThreadCtxSwitch( thrd_c, thrd_c->last );
         }
@@ -99 +98 @@
 
 void yield( void ) {
-        // Safety note : This could cause some false positives due to preemption
-      verify( TL_GET( preemption_state.enabled ) );
+        verify( TL_GET( preemption_state ).enabled );
         BlockInternal( TL_GET( this_thread ) );
-        // Safety note : This could cause some false positives due to preemption
-      verify( TL_GET( preemption_state.enabled ) );
+        verify( TL_GET( preemption_state ).enabled );
 }
 
@@ -112 +109 @@
 }
 
-// KERNEL ONLY
 void ThreadCtxSwitch(coroutine_desc* src, coroutine_desc* dst) {
         // set state of current coroutine to inactive
@@ -120 +116 @@
         // set new coroutine that the processor is executing
         // and context switch to it
-        kernelTLS.this_coroutine = dst;
+        TL_SET( this_coroutine, dst );
         assert( src->stack.context );
         CtxSwitch( src->stack.context, dst->stack.context );
-        kernelTLS.this_coroutine = src;
+        TL_SET( this_coroutine, src );
 
         // set state of new coroutine to active