RFR(s): 8151283: Implement setting jtreg @requires property vm.isG1Supported.
stefan.johansson at oracle.com
Mon Jun 13 07:46:30 UTC 2016
I don't know the code and the way this is invoked good enough to do a
proper review, but I have a style comment. Wouldn't it be nice to only
call the value "vm.gc.G1", and have:
I feel that "acceptable" provide very little information and what we
want to @require is that the test is only run with G1 (when supported)
and I think that is clear from such syntax. Comments?
On 2016-06-10 18:04, Dmitry Fazunenko wrote:
> I'm looking for a couple of Reviewers for a simple change, which
> allows to skip tests depending on non supported garbage collectors.
> The fix introduces four new requires boolean properties:
> Which are set by the jtreg plugin prior the test execution starts.
> After that fix we will be able to replace in tests:
> @requires vm.gc == null | vm.gc == "G1"
> @requires vm.gc.G1.acceptable
> 'vm.gc.G1.acceptable' will be evaluated to 'false' not only if other
> GC is set externally, but also if G1 is not supported.
> In other words the fix allows not to run GC specific tests in not
> applicable configurations.
More information about the hotspot-gc-dev