RFR(S/T) : 8242311 : use reproducible random in hotspot runtime tests

[Igor Ignatyev]

thanks Ioi, pushed!

-- Igor

> On Apr 13, 2020, at 11:14 AM, Ioi Lam <ioi.lam at oracle.com> wrote:
> 
> Hi Igor, the changes look good to me.
> 
> Thanks
> - Ioi
> 
> On 4/13/20 10:34 AM, Igor Ignatyev wrote:
>> @Misha,
>> thanks for the review!
>> 
>> @alias,
>> can I get a 2nd Review?
>> 
>> Cheers,
>> -- Igor
>> 
>>> On Apr 8, 2020, at 11:53 AM, mikhailo.seledtsov at oracle.com wrote:
>>> 
>>> Looks good,
>>> 
>>> Misha
>>> 
>>> On 4/8/20 11:13 AM, Igor Ignatyev wrote:
>>>> http://cr.openjdk.java.net/~iignatyev//8242311/webrev.00
>>>>> 35 lines changed: 12 ins; 1 del; 22 mod;
>>>> Hi all,
>>>> 
>>>> could you please review the patch which updates hotspot runtime tests in the same way as 8242310[1,2] updates compiler tests:
>>>>> marks ... tests w/ randomness k/w and uses Utils.getRandomInstance() instead of Random w/ _random_ seeds where possible? To identify tests which should be marked, I've used both static (in a form of analyzing classes which directly or indirectly depend on Random/SecureRandom/ThreadLocalRandom) and dynamic (by instrumenting the said classes to log tests which called their 'next' methods) analyses. I've decided *not* to mark tests which use SecureRandom only via File.createTemp* b/c in all such cases temp files are not used as a source of randomness, but rather just a reliable way to get a new/empty file/dir.
>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8242311
>>>> webrev: http://cr.openjdk.java.net/~iignatyev/8242311/webrev.00
>>>> 
>>>> NB the patch depends on 8241707[3], which is currently under review[4].
>>>> 
>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8242310
>>>> [2] https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-April/037828.html
>>>> [3] https://bugs.openjdk.java.net/browse/JDK-8241707
>>>> [4] https://mail.openjdk.java.net/pipermail/hotspot-dev/2020-April/041300.html
>>>> 
>>>> Thanks,
>>>> -- Igor
>>>> 
>