On Mon, 22 Mar 2021 19:40:19 GMT, Aleksei Voitylov <avoitylov@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.
Seems reasonable. Though I hadn't realized CDS intruded into the Java layer that way! ------------- Marked as reviewed by dholmes (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/3131