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

Hohensee, Paul hohensee at amazon.com
Fri Oct 18 15:21:33 UTC 2019

Ping again, please.



On 9/25/19, 5:59 PM, "jdk8u-dev on behalf of Hohensee, Paul" <jdk8u-dev-bounces at openjdk.java.net on behalf of hohensee at amazon.com> wrote:

    Ping (it's been nearly a month :)).
    I posted an updated webrev at
    The only difference between it and the previous one is that the copyright dates are 2018 instead of 2019 to match the original patch date.
    Passes tier1 and the modified jtreg test (as part of test/jdk/java/lang/ProcessBuilder) on both linxux-x64 and windows-x64.
    On 8/29/19, 6:30 PM, "jdk8u-dev on behalf of Guo, James" <jdk8u-dev-bounces at openjdk.java.net on behalf of junguoj at amazon.com> wrote:
        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