RFR: 8349400: Improve startup speed via eliminating nested classes

[Johannes Graham]

On Sat, 8 Feb 2025 09:26:29 GMT, Shaojin Wen <swen at openjdk.org> wrote:

>>> The reproducer you provided doesn't try to load KnownOIDs on my machine. What am I missing?
>>> 
>>> Other than that, the change looks reasonable to me.
>> 
>> Me neither and I'm using linux. Are you sure the Startup.java is right?
>> Just curious, do you have a percentage on how much performance difference you observe with this? How about differences in memory usage, is there any?
>
>> > The reproducer you provided doesn't try to load KnownOIDs on my machine. What am I missing?
>> > Other than that, the change looks reasonable to me.
>> 
>> Me neither and I'm using linux. Are you sure the Startup.java is right? Just curious, do you have a percentage on how much performance difference you observe with this? How about differences in memory usage, is there any?
> 
> You are right. Class KnownOIDs are loaded during startup because it is started in IntelliJ IDE and there is an agent.
> 
> 
> git/jdk/build/macosx-aarch64-server-fastdebug/images/jdk/bin/java -verbose:class -javaagent:/Applications/IntelliJ IDEA.app/Contents/lib/idea_rt.jar=58735:/Applications/IntelliJ IDEA.app/Contents/bin
> 
> 
> Therefore, this is not a problem with the default JVM startup parameters, but a problem with the Agent scenario.

@wenshao Are you interested in re-opening this ?

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

PR Comment: https://git.openjdk.org/jdk/pull/23411#issuecomment-2847738889