RFR 9: 8176272 : (process) ProcessHandle::onExit fails to wait for non-child process

[Roger Riggs]

Hi Thomas,

Good idea.  Though it is unlikely that the pid would be re-used between 
the checks of isAlive
but that will remove any window.

Updated webrev:
http://cr.openjdk.java.net/~rriggs/webrev-processhandle-onexit-8176272/index.html

Thanks, Roger


On 3/15/2017 4:19 AM, Thomas Stüfe wrote:
> Hi Roger,
>
> when using isAlive0, would it may make sense to - on the first 
> invocation - remember the process start time and on subsequent 
> invocations to check this time against the new return value? That way 
> you could check for process identity in the case of recycled process ids.
>
> Kind Regards,Thomas
>
> On Tue, Mar 14, 2017 at 9:36 PM, Roger Riggs <roger.riggs at oracle.com 
> <mailto:roger.riggs at oracle.com>> wrote:
>
>     Please review a change to the implementation of
>     ProcessHandle.onExit to
>     correctly handle waiting for any process to terminate and a
>     corresponding new test.
>     On Linux, waitpid only waits for direct children; waiting for
>     non-children
>     now polls for termination.
>
>     Please review and comment:
>
>     Webrev:
>     http://cr.openjdk.java.net/~rriggs/webrev-processhandle-onexit-8176272/index.html
>     <http://cr.openjdk.java.net/%7Erriggs/webrev-processhandle-onexit-8176272/index.html>
>
>     Issue:
>     https://bugs.openjdk.java.net/browse/JDK-8176272
>     <https://bugs.openjdk.java.net/browse/JDK-8176272>
>
>     Thanks, Roger
>
>
>
>