RFR (M) 8209946 [TESTBUG] CDS tests should use "@run driver"

Jiangli Zhou jiangli.zhou at oracle.com
Mon Oct 1 20:50:57 UTC 2018


Hi Ioi,

The change seems ok. Could you please give some details on the benefit 
of using @run driver? Could you also please verify it does not affect 
the jtreg options passed to the dump & run processes for each individual 
test?

Thanks,

Jiangli


On 10/1/18 10:18 AM, Ioi Lam wrote:
> Hi,
>
> Please review this cleanup. It touches a lot of files, but the changes 
> are
> simple and were done by a script (see bug report).
>
> https://bugs.openjdk.java.net/browse/JDK-8209946
> http://cr.openjdk.java.net/~iklam/jdk12/8209946-cds-test-use-run-driver.v01/ 
>
>
> Most CDS tests are currently using "@run main" to run the main class.
> However, the main class is usually just a "script" that prepares JAR 
> files and
> launches child JVM processes that do the real testing (create or use CDS
> archives). Usually, the main class itself is not the subject of the test.
>
> According to http://openjdk.java.net/jtreg/faq.html, we should use 
> "@run driver".
>
> I am validating the changes with hs-tier{1,2,3} tests.
>
> Thanks
> - Ioi



More information about the hotspot-runtime-dev mailing list