Changeset 443 for trunk/kernel/syscalls/sys_wait.c

Timestamp:

May 16, 2018, 4:15:22 PM (7 years ago)

Author:

alain

Message:

Fix few bugs whike debugging the sort multi-thread application.

File:

• trunk/kernel/syscalls/sys_wait.c (modified) (3 diffs)

trunk/kernel/syscalls/sys_wait.c

@@ -73 +73 @@
         }
 
-    // get process owner cluster and calling thread trdid
+    // get calling process owner cluster
     cxy_t   owner_cxy = CXY_FROM_PID( pid );
-    trdid_t trdid     = this->trdid;
 
-    // wait must be executed by the main thread
+    // get calling thread trdid
+    trdid_t trdid = this->trdid;
+
+    // this function must be executed by the process main thread
     if( (owner_cxy != local_cxy) || (LTID_FROM_TRDID(trdid) != 0) )
     {
@@ -113 +115 @@
             child_state = (int)hal_remote_lw ( XPTR(child_cxy,&child_ptr->term_state));
 
-            // test if child process is terminated,
+            // test if this child process is terminated,
             // but termination not yet reported to parent process
             if( ((child_state & PROCESS_TERM_EXIT)  || 
@@ -120 +122 @@
                  ((child_state & PROCESS_TERM_WAIT) == 0) )
             {
-                // get pointer on main thread and PID from child owner process
+                // get pointer on child main thread and PID from child owner process
                 child_pid    = (pid_t)     hal_remote_lw (XPTR( child_cxy , &child_ptr->pid ));
                 child_thread = (thread_t *)hal_remote_lpt(XPTR( child_cxy ,
                                                                 &child_ptr->th_tbl[0] ));
 
-                // set the PROCESS_FLAG_WAIT in owner child descriptor
+                // set the PROCESS_FLAG_WAIT in owner child process descriptor
                 hal_remote_atomic_or( XPTR( child_cxy , &child_ptr->term_state ),
                                       PROCESS_TERM_WAIT );