Changeset e873838 for libcfa/src/concurrency/kernel_private.hfa

Timestamp:

Nov 2, 2020, 12:44:43 PM (4 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, master, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

58688bf, 82f791f

Parents:

f7136f7

Message:

Removed unpark and added support for unpark from the kernel (removing the distinction between the two

File:

• libcfa/src/concurrency/kernel_private.hfa (modified) (5 diffs)

libcfa/src/concurrency/kernel_private.hfa

@@ -33 +33 @@
 }
 
-void __schedule_thread( struct __processor_id_t *, $thread * )
+void __schedule_thread( $thread * )
 #if defined(NDEBUG) || (!defined(__CFA_DEBUG__) && !defined(__CFA_VERIFY__))
-        __attribute__((nonnull (2)))
+        __attribute__((nonnull (1)))
 #endif
 ;
@@ -63 +63 @@
 )
 
-// KERNEL ONLY unpark with out disabling interrupts
-void __unpark( struct __processor_id_t *, $thread * thrd );
-
 #define TICKET_BLOCKED (-1) // thread is blocked
 #define TICKET_RUNNING ( 0) // thread is running
 #define TICKET_UNBLOCK ( 1) // thread should ignore next block
-
-static inline bool __post(single_sem & this, struct __processor_id_t * id) {
-        for() {
-                struct $thread * expected = this.ptr;
-                if(expected == 1p) return false;
-                if(expected == 0p) {
-                        if(__atomic_compare_exchange_n(&this.ptr, &expected, 1p, false, __ATOMIC_SEQ_CST, __ATOMIC_SEQ_CST)) {
-                                return false;
-                        }
-                }
-                else {
-                        if(__atomic_compare_exchange_n(&this.ptr, &expected, 0p, false, __ATOMIC_SEQ_CST, __ATOMIC_SEQ_CST)) {
-                                __unpark( id, expected );
-                                return true;
-                        }
-                }
-        }
-}
 
 //-----------------------------------------------------------------------------
@@ -201 +180 @@
 // Reader side : acquire when using the ready queue to schedule but not
 //  creating/destroying queues
-static inline void ready_schedule_lock( struct __processor_id_t * proc) with(*__scheduler_lock) {
-        unsigned iproc = proc->id;
-        /*paranoid*/ verify(data[iproc].handle == proc);
+static inline void ready_schedule_lock(void) with(*__scheduler_lock) {
+        /*paranoid*/ verify( kernelTLS.this_proc_id );
+
+        unsigned iproc = kernelTLS.this_proc_id->id;
+        /*paranoid*/ verify(data[iproc].handle == kernelTLS.this_proc_id);
         /*paranoid*/ verify(iproc < ready);
 
@@ -225 +206 @@
 }
 
-static inline void ready_schedule_unlock( struct __processor_id_t * proc) with(*__scheduler_lock) {
-        unsigned iproc = proc->id;
-        /*paranoid*/ verify(data[iproc].handle == proc);
+static inline void ready_schedule_unlock(void) with(*__scheduler_lock) {
+        /*paranoid*/ verify( kernelTLS.this_proc_id );
+
+        unsigned iproc = kernelTLS.this_proc_id->id;
+        /*paranoid*/ verify(data[iproc].handle == kernelTLS.this_proc_id);
         /*paranoid*/ verify(iproc < ready);
         /*paranoid*/ verify(data[iproc].lock);
@@ -239 +222 @@
 
 #ifdef __CFA_WITH_VERIFY__
-        static inline bool ready_schedule_islocked( struct __processor_id_t * proc) {
+        static inline bool ready_schedule_islocked(void) {
+                /*paranoid*/ verify( kernelTLS.this_proc_id );
+                __processor_id_t * proc = kernelTLS.this_proc_id;
                 return __scheduler_lock->data[proc->id].owned;
         }