RFR(S/T) : 8242311 : use reproducible random in hotspot runtime tests
igor.ignatyev at oracle.com
Wed Apr 8 18:13:28 UTC 2020
> 35 lines changed: 12 ins; 1 del; 22 mod;
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.
NB the patch depends on 8241707, which is currently under review.
More information about the hotspot-runtime-dev