RFR(L) 8136930: Simplify use of module-system options by custom launchers
Coleen Phillimore
coleen.phillimore at oracle.com
Wed Jul 20 21:47:15 UTC 2016
http://cr.openjdk.java.net/~hseigel/bug_8136930.hs/src/share/vm/runtime/arguments.cpp.udiff.html
To check if is_internal_module_property, the numbered ones go to
is_matching_numbered_property, but do you also want to disallow:
jdk.module.addreads.foo or jdk.module.addreads (with no number?)
Why aren't addmods and limitmods allowed be specified more than once on
the command line and handled like the other ones?
+ // options. For example: use "--add-modules java.sql", not
"-Djdk.module.addmods=java.sql"
Does this comment need an equal sign?
http://cr.openjdk.java.net/~hseigel/bug_8136930.hs/test/compiler/unsafe/UnsafeGetConstantField.java.udiff.html
Can you fix the indentation?
We should add an option -XX:+UseUnsafe to be a shortcut for this:
--add-exports=java.base/jdk.internal.misc=ALL-UNNAMED
The rest looks good. This implementation looks very clean.
Thanks,
Coleen
On 7/17/16 7:05 PM, harold seigel wrote:
> Hi,
>
> Please review these Hotspot VM only changes to process the seven
> module-specific options that have been renamed to have gnu-like
> names. JDK changes for this bug will be reviewed separately.
>
> Descriptions of these options are here
> <http://openjdk.java.net/jeps/293>. For these six options,
> --module-path, --upgrade-module-path, --add-modules, --limit-modules,
> --add-reads, and --add-exports, the JVM just sets a system property.
> For the --patch-module option, the JVM sets a system property and then
> processes the option in the same way as when it was named -Xpatch.
>
> Additionally, the JVM now checks properties specified on the command
> line. If a property matches one of the properties used by one of the
> above options then the JVM ignores the property. This forces users to
> use the explicit option when wanting to do things like add a module or
> a package export.
>
> The RFR contains two new tests. Also, many existing tests were
> changed to use the new option names.
>
> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8136930
>
> Webrev: http://cr.openjdk.java.net/~hseigel/bug_8136930.hs/
>
> The changes were tested with the JCK lang and VM tests, the JTreg
> hotspot tests, and the RBT hotspot nightlies.
>
> Thanks, Harold
More information about the hotspot-runtime-dev
mailing list