[lworld] RFR: 8376221: [lworld] Do not store array of InlineLayoutInfo for all InstanceKlasses [v5]

Frederic Parain fparain at openjdk.org
Thu Jan 29 21:03:31 UTC 2026


On Wed, 28 Jan 2026 12:08:57 GMT, Paul Hübner <phubner at openjdk.org> wrote:

>> src/hotspot/share/classfile/classFileParser.cpp line 6298:
>> 
>>> 6296:           if (klass != nullptr) {
>>> 6297:             if (klass->is_inline_klass()) {
>>> 6298:               set_inline_layout_info_klass(fieldinfo.index(), klass, CHECK);
>> 
>> I think this is a property already precent in the code but this change made me reflect on this. 
>> 
>> What are the expected interactions for pre-loaded classes if we fail to load parse the class file which contains them. For example here if we have just resolved the pre-loaded class, and then get metaspace out of memory and stop loading this class file. 
>> 
>> Is this the correct (specification wise) interaction for partially failed class file parsing?
>
> There's nothing from the specification point of view that enforces that a class and its pre-loaded (via `LoadableDescriptor`) classes must be all-or-nothing. 
>> Classes and interfaces mentioned by LoadableDescriptors may optionally be loaded when the referencing class is derived and created ([5.3.5](https://cr.openjdk.org/~dlsmith/jep401/jep401-20250926/specs/value-objects-jvms.html#jvms-5.3.5)), or during early stages of linking ([5.4](https://cr.openjdk.org/~dlsmith/jep401/jep401-20250926/specs/value-objects-jvms.html#jvms-5.4)).

This is the same situation as when the JVM is loading a class A, and during the loading of A, it successfully loads A's super-class B, but then encounter an issue while continuing the loading of A (OOM in Metaspace, or B is an interface, or B is final etc.). The loading of A fails, but class B remains in the system dictionary, until its class loader is unloaded.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1966#discussion_r2743530911


More information about the valhalla-dev mailing list