RFR: 8349400: Improve startup speed via eliminating nested classes [v2]

Roger Riggs rriggs at openjdk.org
Mon Apr 14 14:43:00 UTC 2025


On Sat, 5 Apr 2025 01:30:49 GMT, Shaojin Wen <swen at openjdk.org> wrote:

>> During JVM startup, the class KnownOIDs is loaded. KnownOIDs has 10 anonymous classes, which slows down the startup. This PR is to improve KnownOIDs and eliminate unnecessary embedded classes.
>> 
>> 
>> Here's how to reproduce this:
>> 
>> 
>> public class Startup {
>>     public static void main(String[] args) {}
>> }
>> 
>> 
>> 
>> java -verbose:class Startup
>> 
>> 
>> 
>> [0.665s][info][class,load] sun.security.util.KnownOIDs
>> [0.666s][info][class,load] sun.security.util.KnownOIDs$1
>> [0.667s][info][class,load] sun.security.util.KnownOIDs$2
>> [0.667s][info][class,load] sun.security.util.KnownOIDs$3
>> [0.668s][info][class,load] sun.security.util.KnownOIDs$4
>> [0.668s][info][class,load] sun.security.util.KnownOIDs$5
>> [0.668s][info][class,load] sun.security.util.KnownOIDs$6
>> [0.668s][info][class,load] sun.security.util.KnownOIDs$7
>> [0.669s][info][class,load] sun.security.util.KnownOIDs$8
>> [0.669s][info][class,load] sun.security.util.KnownOIDs$9
>> [0.669s][info][class,load] sun.security.util.KnownOIDs$10
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   from @valeriepeng

Looks ok; but will need to be re-opened. It would seem odd to approve a closed PR.

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

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


More information about the security-dev mailing list