<div dir="ltr">Just a follow-up.  It seems like there are three use cases for the StartMonitoringChildProcess code-path (poorly named, since FreeBSD and linux process plugins also have a ProcessMonitor class leading to great confusion).<div><br></div><div>1) Some places want to Join() on a process exiting.  They currently do this by calling Join() on a HostThread returned by StartMonitoringChildProcess, but the important thing is just that they want to join, not how it's implemented.</div><div><br></div><div>2) Processes need to be reaped when they exit so as not to leave zombie processes.</div><div><br></div><div>3) Some places want to have a user-specified callback executed asynchronously in an arbitrary thread context when a process exits.  </div><div><br></div><div>Does this cover everything?  There's currently alot of built in assumptions about which platforms want what subset of the above functionality, but I think I'm starting to get a pretty good handle on this code and have a good idea of how to restructure it to be more platform-agnostic.  Want to make sure I understand all the primary use cases first though.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 16, 2014 at 11:55 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The last major piece of the Host layer I'd like to address is process launching and cleanup.  From looking over the code it seems we have the following different ways to launch a process:<div><br></div><div>MacOSX:</div><div>    * Applescript</div><div>    * XPC</div><div>    * posix_spawn</div><div><br></div><div>Other posix variants:</div><div>    * posix_spawn</div><div><br></div><div>Windows:</div><div>    * Native windows launcher</div><div><br></div><div><br></div><div>Among these, there are a couple of different ways to reap processes on exit and/or "join" on them.  These are:</div><div><br></div><div><div>MacOSX:</div><div>    * Applescript    [ No process reaping or monitoring occurs. ]</div><div>    * XPC               [ Uses MacOSX-specific dispatch library for monitoring ]</div><div>    * posix_spawn [ Uses MacOSX-specific dispatch library for monitoring ]</div><div><br></div><div>Other posix variants:</div><div>    * posix_spawn   [ Launches a background thread to monitor for process exit, join on the thread to join on the process ]</div><div><br></div><div>Windows:</div><div>    * Native windows launcher  [ WaitForSingleObject ]</div></div><div><br></div><div>A few questions: </div><div><br></div><div>1) Is Join() on a process a useful operation that people would be interested in seeing implemented for all platforms?</div><div><br></div><div>2) On Linux at least, if you don't waitpid() on a process you'll end up with zombies.  It seems this is true on MacOSX as well, because I see waitpid() in StartMonitoringChildProcess.  Is this not true for the Applescript launcher?  Why doesn't the Applescript launching code path call StartMonitoringChildProcess() anywhere?</div><div><br></div><div>3) Speaking of the Applescript launcher, what is it and what is it used for?</div><div><br></div><div><br></div><div>I'll probably have more questions as I wrap my head around this a little more.  Thanks</div></div>
</blockquote></div><br></div>