RFR: 8263968: CDS: java/lang/ModuleLayer.EMPTY_LAYER should be singleton

Ioi Lam iklam at openjdk.java.net
Mon Mar 22 20:32:39 UTC 2021


On Mon, 22 Mar 2021 19:40:19 GMT, Aleksei Voitylov <avoitylov at openjdk.org> wrote:

> With CDS and G1, test api/java_lang/ModuleLayer/Boot.html fails in JDK 16. 
> 
> After JDK-8253081 with CDS enabled two instances of empty layer appear in the runtime. One comes from default initialization of java/lang/ModuleLayer.EMPTY_LAYER, another one comes from CDS archive and is returned by ModuleLayer.boot().parents().get(0). The fix makes java/lang/ModuleLayer.EMPTY_LAYER a singleton with both CDS on and off, similar to java/lang/module/Configuration.EMPTY_CONFIGURATION.
> 
> Testing: JCK 16, jtreg (including newly added test), pre-submit GitHub actions tests.

LGTM. Thanks for fixing this!

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

Marked as reviewed by iklam (Reviewer).

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


More information about the hotspot-runtime-dev mailing list