RFR (xs) 8251209 [TESTBUG] CDS jvmti tests should use "@modules java.instrument"

David Holmes david.holmes at oracle.com
Thu Aug 6 21:52:59 UTC 2020


Hi Ioi,

On 7/08/2020 4:25 am, Ioi Lam wrote:
> https://bugs.openjdk.java.net/browse/JDK-8251209
> http://cr.openjdk.java.net/~iklam/jdk16/8251209-cds-jvmti-tests-modules-tag.v01/ 
> 
> 
> Summary -- changed the tests from (mis)using
> 
>   * @requires vm.flavor != "minimal"
> 
> to
> 
>   * @modules java.instrument
> 
> ... to be consistent with other jvmti tests.

That seems like an invalid precondition to me. It would have been 
somewhat valid in the Compact Profiles world when we did not provide 
"java.instrument" in the profiles which supported MinimalVM, but you can 
define a minimal VM in a build that still has all modules available. I 
don't think building the minimal VM makes any changes to the supported 
modules.

Also AIUI the @modules statement simply adds the necessary command-line 
args to use the java.instrument module (if present), it doesn't ensure 
that the listed module has to be present.

David

> Thanks
> - Ioi


More information about the hotspot-runtime-dev mailing list