drop lock of process_map before yielding to other tasks

syswonder · Sep 27, 2024 · 2431ea7 · 2431ea7
1 parent e0cbc0f
commit 2431ea7
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/api/ruxos_posix_api/src/imp/task.rs b/api/ruxos_posix_api/src/imp/task.rs
@@ -78,11 +78,13 @@ pub fn sys_wait4(
                 } else if options & WNOHANG != 0 {
                     return 0; // No child process
                 }
-                // for single-cpu system, we must yield to other tasks instead of dead-looping here.
-                yield_now();
             } else {
                 return -1; // No such process
             }
+            // drop lock before yielding to other tasks
+            drop(process_map);
+            // for single-cpu system, we must yield to other tasks instead of dead-looping here.
+            yield_now();
         }
     } else if pid == -1 {
         let mut to_remove: Option<u64> = None;
@@ -104,6 +106,8 @@ pub fn sys_wait4(
             if options & WNOHANG != 0 {
                 return 0; // No child process
             }
+            // drop lock before yielding to other tasks
+            drop(process_map);
             // for single-cpu system, we must yield to other tasks instead of dead-looping here.
             yield_now();
         }