RFR: 8256487: Handle disableEagerInitialization for archived lambda proxy classes

Yumin Qi minqi at openjdk.java.net
Thu Nov 19 19:46:08 UTC 2020


On Wed, 18 Nov 2020 23:58:25 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Before this change, the setting of the `jdk.internal.lambda.disableEagerInitialization` property was not captured during dumping of lambda proxy classes. There's a workaround in `LambdaProxyClassArchive.find`, it won't call `findFromArchive` if the above property is set.
> 
> This change adds handling of the `jdk.internal.lambda.disableEagerInitialization` property, specifically:
> 
> - remove the above workaround;
> 
> - capture the setting of the property in the archive header during CDS dump time;
> 
> - during runtime when finding an archived lambda proxy class, the setting of the property will be compared with the stored value in the archive header.
> If the values don't match, the archived lambda proxy class won't be used.
> 
> Tests:
> 
> - [x] ran all cds tests locally on linux-x64
> 
> - [x] ran the `hotspot_appcds_dynamic` test group with `-Dtest.dynamic.cds.archive=true` on linux-x64
> 
> - [x] mach5 tiers 1,2,3 (in progress)

src/hotspot/share/memory/metaspaceShared.cpp line 76:

> 74: #endif
> 75: 
> 76: #include <string.h>

This include is strange, usually we do not include std head file in .cpp file.

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

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


More information about the core-libs-dev mailing list