Let reap() communicate the dead process's exit status to the caller.

This way the scheduler doesn't need to plumb the exit status into the waiter.
We still plumb the waitee pid though, I don't love it but it can be fixed.

1 files changed, 6 insertions, 3 deletions

diff --git a/Kernel/Scheduler.cpp b/Kernel/Scheduler.cpp
index 989ed0cfbb..c2471dfbe7 100644
--- a/Kernel/Scheduler.cpp
+++ b/Kernel/Scheduler.cpp
@@ -39,7 +39,6 @@ bool Scheduler::pick_next()
                 if (child.state() != Process::Dead)
                     return true;
                 if (process.waitee() == -1 || process.waitee() == child.pid()) {
-                    process.m_waitee_status = (child.m_termination_status << 8) | child.m_termination_signal;
                     process.m_waitee = child.pid();
                     process.unblock();
                     return false;
@@ -75,8 +74,12 @@ bool Scheduler::pick_next()
         }
 
         if (process.state() == Process::Dead) {
-            if (current != &process && !Process::from_pid(process.ppid()))
-                Process::reap(process);
+            if (current != &process && !Process::from_pid(process.ppid())) {
+                auto name = process.name();
+                auto pid = process.pid();
+                auto exit_status = Process::reap(process);
+                kprintf("reaped unparented process %s(%u), exit status: %u\n", name.characters(), pid, exit_status);
+            }
             return true;
         }