RFR(S) : 8242314 : use reproducible random in vmTestbase shared code
Kim Barrett
kim.barrett at oracle.com
Sat Apr 25 23:57:46 UTC 2020
> On Apr 22, 2020, at 2:03 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
>
> http://cr.openjdk.java.net/~iignatyev/8242314/webrev.00/
>> 122 lines changed: 71 ins; 19 del; 32 mod;
>
>
> Hi all,
>
> could you please review this small patch which updates shared vmTestbase test code to use j.t.l.Utils.getRandomInstance() as a random number generator or as a source of seed values? this patch lays the ground work for updates of :vmTestbase_* test groups which are to be done by remaining subtasks of 8241623[1], such split make it possible to work on/review/test testgroup-specific changes in concurrently.
>
> unfortunately, such extra dependency on /test/lib classes broke the shaky balance and caused some tests to fail due to "split testlibrary" problem (see 8188828[2] for the definition), the solution to that is to remove all dependencies on /test/lib classes from ExecDriver class, so the patch also includes "inlining" of several jdk.test.lib.Utils and Platform methods/fields into ExecDriver.
>
> webrev: http://cr.openjdk.java.net/~iignatyev/8242314/webrev.00/
> testing: all :vmTestbase_* test groups on {linux,windows,macosx}-x64 and solaris-sparcv9
> JBS: https://bugs.openjdk.java.net/browse/JDK-8242314
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8241623
> [2] https://bugs.openjdk.java.net/browse/JDK-8188828
>
> Thanks,
> -- Igor
The test changes look fine.
But I don't feel like I understand the "split library" problem well
enough to review the ExecDriver changes. I will say that I don't like
the solution of copying code, and hope there is a better way. Is
there a bug for the problem? It seems like it could crop up again,
perhaps repeatedly, in which case the code copying becomes increasingly
bad. And it might be a good idea to leave some breadcrumbs associated
with this code copying workaround, so an eventual better fix could also
cleanup the duplication.
More information about the hotspot-dev
mailing list