RFR(M): 8044186 Introduce a reproducible random generator

Igor Ignatyev igor.ignatyev at oracle.com
Fri Sep 19 21:03:00 UTC 2014


Hi Sergei,

test/testlibrary/com/oracle/java/testlibrary/Utils.java:

don't you need an additional check RANDOM_GENERATOR == null after line 
#355 to prevent double creating of Random which leads to the problems w/ 
reproducing on MT environments?

Thanks,
Igor

On 09/19/2014 05:57 PM, Sergei Kovalev wrote:
> Hi All,
>
> I'm working for CR https://bugs.openjdk.java.net/browse/JDK-8044186 that
> submitted against tests.
>
> There's the several tests which use randomly generated numbers. It's
> needed to introduce a way that provide a common manner to reproduce the
> same random sequence, set seed.
>
> There are two possible way to provide seed value:
> - system property
> - command line arguments.
>
> I chose system property to provide seed value because it could be
> obtained in any part of code without additional effort. In case passing
> seed value as command line argument we have to introduce additional
> logic (and unwanted code) to pass this value in appropriate method.
>
> I introduced factory method getRandomInstance in the file Utility.java
> and make reference to this method in all applicable tests.
> There is other utility class: Verifier.java. It used in several other
> testcases from test/compiler/intrinsics/mathexact/ package. Therefore
> many files contains modifications only for jtreg annotations.
>
> I've tested all related tests locally. All tests passed and seed value
> printed to the logs.
>
> links:
> bug: https://bugs.openjdk.java.net/browse/JDK-8044186
> webrev: http://cr.openjdk.java.net/~vlivanov/kovalev/8044186/webrev.00/
>


More information about the hotspot-compiler-dev mailing list