[8u] RFR 8208715: Conversion of milliseconds to nanoseconds in UNIXProcess contains bug
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 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 and add a millisElapsedSince(long startNanoTime) method for it.
Testing: x86_64 build, affected tests , tier1
More information about the jdk8u-dev