Changeset 89a3df5 for src/libcfa/concurrency

Timestamp:

Mar 15, 2017, 3:59:09 PM (8 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

0e7b95c

Parents:

0ea1b65

Message:

Removed unnecessary getter for this_processor

Location:

src/libcfa/concurrency

Files:

• coroutine.c (modified) (2 diffs)
• kernel.c (modified) (2 diffs)
• thread.c (modified) (4 diffs)

src/libcfa/concurrency/coroutine.c

@@ -32 +32 @@
 #include "invoke.h"
 
-extern processor * get_this_processor();
+extern thread_local processor * this_processor;
 
 //-----------------------------------------------------------------------------
@@ -109 +109 @@
 
         // set new coroutine that task is executing
-        get_this_processor()->current_coroutine = dst;                  
+        this_processor->current_coroutine = dst;
 
         // context switch to specified coroutine

src/libcfa/concurrency/kernel.c

@@ -55 +55 @@
 thread_local processor * this_processor;
 
-processor * get_this_processor() {
-        return this_processor;
-}
-
 coroutine * this_coroutine(void) {
         return this_processor->current_coroutine;
@@ -317 +313 @@
 
 void ScheduleInternal( spinlock * lock ) {
-        get_this_processor()->finish.action_code = Release;
-        get_this_processor()->finish.lock = lock;
+        this_processor->finish.action_code = Release;
+        this_processor->finish.lock = lock;
         suspend();
 }
 
 void ScheduleInternal( thread * thrd ) {
-        get_this_processor()->finish.action_code = Schedule;
-        get_this_processor()->finish.thrd = thrd;
+        this_processor->finish.action_code = Schedule;
+        this_processor->finish.thrd = thrd;
         suspend();
 }
 
 void ScheduleInternal( spinlock * lock, thread * thrd ) {
-        get_this_processor()->finish.action_code = Release_Schedule;
-        get_this_processor()->finish.lock = lock;
-        get_this_processor()->finish.thrd = thrd;
+        this_processor->finish.action_code = Release_Schedule;
+        this_processor->finish.lock = lock;
+        this_processor->finish.thrd = thrd;
         suspend();
 }

src/libcfa/concurrency/thread.c

@@ -28 +28 @@
 }
 
-extern processor * get_this_processor();
+extern thread_local processor * this_processor;
 
 //-----------------------------------------------------------------------------
@@ -77 +77 @@
         thread*  thrd_h = get_thread   (this);
         thrd_c->last = this_coroutine();
-        get_this_processor()->current_coroutine = thrd_c;
+        this_processor->current_coroutine = thrd_c;
 
         LIB_DEBUG_PRINTF("Thread start : %p (t %p, c %p)\n", this, thrd_c, thrd_h);
@@ -94 +94 @@
 
 void yield( void ) {
-        ScheduleInternal( get_this_processor()->current_thread );
+        ScheduleInternal( this_processor->current_thread );
 }
 
@@ -107 +107 @@
         // set new coroutine that the processor is executing
         // and context switch to it
-        get_this_processor()->current_coroutine = dst;  
+        this_processor->current_coroutine = dst;
         CtxSwitch( src->stack.context, dst->stack.context );
-        get_this_processor()->current_coroutine = src;  
+        this_processor->current_coroutine = src;
 
         // set state of new coroutine to active