RFR(S) : 8181053 : port basicvmtest to jtreg
Erik Helin
erik.helin at oracle.com
Wed Jun 14 07:41:57 UTC 2017
On 06/14/2017 01:09 AM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8181053/webrev.00/index.html
>> 121 lines changed: 54 ins; 67 del; 0 mod;
>
> Hi all,
>
> could you please review this small patch which introduces jtreg version of basicvmtest test?
>
> make version of basicvmtest also included sanity testing for CDS on client JVM, but this testing modified the product binaries, so it might interfere with results of other tests and is not very reliable. I have consulted w/ Misha about this, and he assured me that there are other better CDS tests which check the same functionality, so we should not lose test coverage there.
>
> webrev: http://cr.openjdk.java.net/~iignatyev//8181053/webrev.00/index.html
Looks good, Reviewed.
Thank you for this patch Igor! I've been meaning to fix this for a long
time but never got around to it...
Erik
> jbs: https://bugs.openjdk.java.net/browse/JDK-8181053
> testing: jprt, new added test
>
> Thanks,
> -- Igor
>
More information about the hotspot-dev
mailing list