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