RFR: 8350982: -server|-client causes fatal exception on static JDK

[David Holmes]

On Tue, 4 Mar 2025 01:43:29 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

> Please review the `Arguments::parse_each_vm_init_arg` change to ignore`-server|-client` options, which avoids unrecognized option error on static JDK. 
> 
> On regular JDK, '-server|-client' options are processed/removed from command-line arguments by `CheckJvmType` during `CreateExecutionEnvironment`. That happens before `Arguments::parse_each_vm_init_arg` is called. With jvm.cfg setting, only server vm is known and client is ignored. So specifying '-server' and '-client' in command-line is really a no-op. 
> 
> On static JDK, the VM is statically linked with the launcher, and `CreateExecutionEnvironment` & `CheckJvmType` are not called. As the result, `Arguments::parse_each_vm_init_arg` could see `-server|-client` when running on static JDK, if the options are specified in the command line.

src/hotspot/share/runtime/arguments.cpp line 2812:

> 2810:       // On regular JDK, '-server|-client' options are processed and removed
> 2811:       // from command-line arguments by CheckJvmType (see
> 2812:       // src/java.base/share/native/libjli/java.c). When running on static JDK,

Why not remove this in the launcher?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23881#discussion_r1978735436