RFR: 8343218: Disable allocating interface and abstract classes in non-class metaspace [v2]

David Holmes dholmes at openjdk.org
Fri Nov 1 12:50:28 UTC 2024


On Thu, 31 Oct 2024 18:31:35 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/oops/instanceKlass.cpp line 460:
>> 
>>> 458:   // are the only ones that require compression and allocation in the limited class metaspace
>>> 459:   // region.  But ... there are C2 dependencies that all klasses are allocated to the class metaspace, so
>>> 460:   // always use the class metaspace until JDK-NNNNNNN is resolved.
>> 
>> JDK-NNNNNN was supposed to be replaced by an actual bug number. :)
>
> I was wondering where the NNNN came from.  I do not want to put the number in the code, and there may be a performance problem that I'm working on.  I may or may not push this.  The c2 problem will be addressed with an assert and I'll look through c1 and see if there may be a similar problem.

I'm left wondering what this is supposed to do. We made changes, then we are disabling them until something is fixed but we are not saying what something that is. ???

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21769#discussion_r1825780949


More information about the hotspot-dev mailing list