[8u] RFR 8208715: Conversion of milliseconds to nanoseconds in UNIXProcess contains bug

Guo, James junguoj at amazon.com
Fri Aug 30 01:29:06 UTC 2019


Original bug:
Original patch does not apply cleanly to 8u:
        1. java.base/unix doesn't exist. I had to move the change of java.base/unix/classes/java/lang/ProcessImpl.java 
to solaris/classes/java/lang/UNIXProcess.java to make the patch work in Unix.
        2. Due to the conflict in test/java/lang/ProcessBuilder/Basic.java, I had to replace the testcase that checks 
Process.waitFor(timeout, TimeUnit.MILLISECONDS) with the one in 12u[1] and add a millisElapsedSince(long startNanoTime) method for it.
8u webrev:
Testing: x86_64 build, affected tests [2], tier1
James Guo
[1] http://hg.openjdk.java.net/jdk/jdk/file/41257a58a588/test/jdk/java/lang/ProcessBuilder/Basic.java#l2410
[2] https://bugs.openjdk.java.net/secure/attachment/78016/JI9056393.java

More information about the jdk8u-dev mailing list