Changeset 593 for trunk/kernel/kern/scheduler.c

Timestamp:

Nov 10, 2018, 2:16:38 PM (5 years ago)

Author:

alain

Message:

Cosmetic: improve debug.

File:

• trunk/kernel/kern/scheduler.c (modified) (7 diffs)

trunk/kernel/kern/scheduler.c

@@ -179 +179 @@
     sched = &core->scheduler;
 
-    // take the lock protecting sheduler state
-    busylock_acquire( &sched->lock );
-
-    ////// scan user threads to handle both ACK and DELETE requests
+    /////////////// scan user threads to handle both ACK and DELETE requests
     root = &sched->u_root;
     iter = root->next;
@@ -215 +212 @@
             // get thread ltid
             ltid = LTID_FROM_TRDID( thread->trdid);
+
+            // take the lock protecting sheduler state
+            busylock_acquire( &sched->lock );
 
             // update scheduler state
@@ -236 +236 @@
             }
 
+            // release the lock protecting sheduler state
+            busylock_release( &sched->lock );
+
 // check th_nr value
 assert( (process->th_nr > 0) , "process th_nr cannot be 0\n" );
@@ -241 +244 @@
             // remove thread from process th_tbl[]
             process->th_tbl[ltid] = NULL;
-            hal_atomic_add( &process->th_nr , - 1 );
+            count = hal_atomic_add( &process->th_nr , - 1 );
  
             // release memory allocated for thread descriptor
             thread_destroy( thread );
+
+            hal_fence();
 
 #if DEBUG_SCHED_HANDLE_SIGNALS
@@ -288 +293 @@
             // get thread ltid
             ltid = LTID_FROM_TRDID( thread->trdid);
+
+            // take the lock protecting sheduler state
+            busylock_acquire( &sched->lock );
 
             // update scheduler state
@@ -309 +317 @@
             }
 
+            // release the lock protecting sheduler state
+            busylock_release( &sched->lock );
+
             // get number of threads in local kernel process
             count = process_zero.th_nr;
@@ -330 +341 @@
         }
     }
-
-    // release the lock protecting sheduler state
-    busylock_release( &sched->lock );
-
 } // end sched_handle_signals()