RFR: 8258917: NativeMemoryTracking is handled by launcher inconsistenly [v2]

[Alex Menkov]

On Sun, 17 Jan 2021 12:55:35 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Non-lava launchers should process all "-J" arguments
>
> Alex,
> 
> This approach results in two scans of the argument list in the IsJavaArgs case. I don't know if we care about startup in the non-Java launchers, but this will likely affect it.
> 
> David

@dholmes-ora 

> Also, I'm not sure the scanning logic in SetJvmEnvironment is valid for
> the IsJavaArgs case. It states:
> 
> /*
> * Since this must be a VM flag we stop processing once we see
> * an argument the launcher would not have processed beyond (such
> * as -version or -h), or an argument that indicates the following
> * arguments are for the application (i.e. the main class name, or
> * the -jar argument).
> */
> 
> but the argument rules for other commands are different.

yes, you are right.
TranslateApplicationArgs translates all "-J" args.
I updated the fix.
For non-java launchers we don't need to scan java args as we know they don't contain -J-XX:NativeMemoryTracking

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

PR: https://git.openjdk.java.net/jdk/pull/2106