<div dir="ltr"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false"><div id="gmail-:33e" class="gmail-Am gmail-aiL gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:376px" aria-controls=":3az" aria-expanded="false">Hi all,<div><br></div><div>I notice the meaning of the test property `vm.gc` is not clear. </div><div>The method `VMProps::vmGCforCDS` (shown below, in directory: test/jtreg-ext/requires/VMProps.java) </div><div>extracts the property `vm.gc` from the option `test.cds.runtime.options`. </div><div>So the current `vm.gc` is about both CDS and GC, not only the GC.<br>I propose changing the current test property name `vm.gc` to `vm.gc.cds` or `vm.cds.gc`.</div><div><div><br class="gmail-Apple-interchange-newline">Waiting for your opinion. Any ideas will be appreciated.</div><div><br></div></div><div>```java<br>    /**<br>     * "jtreg -vmoptions:-Dtest.cds.runtime.options=..." can be used to specify<br>     * the GC type to be used when running with a CDS archive. Set "vm.gc" accordingly,<br>     * so that tests that need to explicitly choose the GC type can be excluded<br>     * with "@requires vm.gc == null".<br>     *<br>     * @param map - property-value pairs<br>     */<br>    protected void vmGCforCDS(SafeMap map) {<br>        if (!GC.isSelectedErgonomically()) {<br>            // The GC has been explicitly specified on the command line, so<br>            // jtreg will set the "vm.gc" property. Let's not interfere with it.<br>            return;<br>        }<br><br>        String jtropts = System.getProperty("test.cds.runtime.options");<br>        if (jtropts != null) {<br>            for (String opt : jtropts.split(",")) {<br>                if (opt.startsWith(GC_PREFIX) && opt.endsWith(GC_SUFFIX)) {<br>                    String gc = opt.substring(GC_PREFIX.length(), opt.length() - GC_SUFFIX.length());<br>                    map.put("vm.gc", () -> gc);<br>                }<br>            }<br>        }<br>    }<br></div><div>```</div><div><br></div><div>Best Regards,</div><div>-- Guoxiong</div></div></div></div></div></div></div></div></div>