[8u-dev] RFA: JDK-8000975 - (process) Merge UNIXProcess.java.bsd & UNIXProcess.java.linux

[Seán Coffey]

Changes look fine to me Peter. Thanks for backporting this one. Please 
close out JDK-8047340 as a duplicate also.

I took the opportunity to run your change through our JPRT build/test 
system and all looks fine.

regards,
Sean.

On 23/07/2014 12:53, Peter Levart wrote:
> Hi,
>
> I would like to backport the fix for:
>
> https://bugs.openjdk.java.net/browse/JDK-8000975
>
> Which seems to also fix the following issue:
>
> https://bugs.openjdk.java.net/browse/JDK-8047340
>
> The discussion thread starts here:
>
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-March/026062.html 
>
>
> The original fix was made for jdk9 and applied to:
>
> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/27e0909d3fa0
> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/27e0909d3fa0
>
> It does not import cleanly into jdk8u-dev repository because a fix to 
> UNIXProcess.java.solaris was applied to jdk8u-dev lately:
>
> http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/9189a6d49ac0
>
> ...which was not applied to jdk9-dev. Since the presented patch 
> supersedes this fix (a fix for Solaris variant to use a thread pool to 
> manage process reaper tasks), nothing is lost.
>
> Here's a webrev of the proposed backported fix:
>
> http://cr.openjdk.java.net/~plevart/jdk8u-dev/UNIXProcess/webrev.01/
>
> You can compare it to the webrev of original jdk9-dev fix:
>
> http://cr.openjdk.java.net/~plevart/jdk9-dev/UNIXProcess/webrev.10/
>
>
> Regards, Peter
>