[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