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

Hohensee, Paul hohensee at amazon.com
Thu Sep 26 00:56:58 UTC 2019


Ping (it's been nearly a month :)).

I posted an updated webrev at

http://cr.openjdk.java.net/~phh/8208715/webrev.00/

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.

Thanks,

Paul

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:

    Hi,
    
    Original bug:
         https://bugs.openjdk.java.net/browse/JDK-8208715
         https://hg.openjdk.java.net/jdk/jdk/rev/41257a58a588
            
    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:
         http://cr.openjdk.java.net/~alvdavi/webrevs/8208715/webrev.8u.00/
            
    Testing: x86_64 build, affected tests [2], tier1
            
    Thanks,
    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