RFR: (s) 8036095: RMI tests using testlibrary.RMID and testlibrary.JavaVM do not pass through vmoptions

Alan Bateman Alan.Bateman at oracle.com
Thu Mar 6 08:29:15 UTC 2014


On 05/03/2014 22:05, Stuart Marks wrote:
> Hi all,
>
> Please review this fix to change RMI's test library to pass through 
> vmoptions and javaoptions to rmid and other JVM subprocesses.
>
> I've marked this as a "small" review because the changes of substance 
> are confined to the first four files in the webrev, and they're pretty 
> small. The remaining changes are to the individual security.policy 
> files of each test, which requires different permissions granted in 
> order to allow reading of the test.vm.opts and test.java.opts properties.
>
> Webrev:
>
>     http://cr.openjdk.java.net/~smarks/reviews/8036095/webrev.0/
I've looked through the changes and they look okay to me. I also grabbed 
the patch and ran the jdk_rmi tests with a few XX options specific to 
jtreg via -vmoption and it works.

Just on David's comment then I think it would be surprising to specify 
options to jtreg but have some but not all tests use it consistently. 
There have been a number of passes over the shell tests to make sure 
that VM options are passed through but there is still work required to 
check tests that use Runtime.exec or ProcessBuilder to start other VMs 
to see that they pass through the expected options. As to whether 
someone would run these tests with -Xcomp or other stress options then I 
don't know but if they do then it would be strange for some tests to use 
it and not others.

-Alan



More information about the core-libs-dev mailing list