Changeset 450 for trunk/kernel/kern/process.c

Timestamp:

Jun 29, 2018, 10:44:14 AM (6 years ago)

Author:

alain

Message:

Fix a bug in function sched_handle_signal():
When the deleted user thread is the last executed thread,
the sched->u_last field must be updated to point on another user thread.

File:

• trunk/kernel/kern/process.c (modified) (4 diffs)

trunk/kernel/kern/process.c

@@ -412 +412 @@
     cluster_process_copies_unlink( process );
 
-    // remove process from children_list if process owner cluster
+    // remove process from children_list 
+    // and release PID if owner cluster
     if( CXY_FROM_PID( pid ) == local_cxy )
     {
@@ -429 +430 @@
             hal_remote_atomic_add( children_nr_xp , -1 );
         remote_spinlock_unlock( children_lock_xp );
-    }
-
-    // release the process PID to cluster manager if process owner cluster
-    if( CXY_FROM_PID( pid ) == local_cxy ) cluster_pid_release( pid );
+
+    // release the process PID to cluster manager
+    cluster_pid_release( pid );
+
+    }
 
     // FIXME close all open files and update dirty [AG]
@@ -1057 +1059 @@
     // remove thread from th_tbl[]
     process->th_tbl[ltid] = NULL;
-    process->th_nr--;
+    process->th_nr = count-1;
 
     // release lock protecting th_tbl
@@ -1065 +1067 @@
     return (count == 1);
 
-}  // process_remove_thread()
+}  // end process_remove_thread()
 
 /////////////////////////////////////////////////////////