RFR: 8290981: Enable CDS for zero builds [v2]
Erik Joelsson
erikj at openjdk.org
Mon Aug 15 12:46:13 UTC 2022
On Sun, 14 Aug 2022 05:52:01 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> Enable CDS for zero builds. `java --version` is about 2x faster now.
>>
>>
>> $ perf stat -r 40 ./images/jdk/bin/java -Xshare:off -version
>> 0.034645 +- 0.000044 seconds time elapsed ( +- 0.13% )
>> $ perf stat -r 40 ./images/jdk/bin/java -version
>> 0.018080 +- 0.000388 seconds time elapsed ( +- 2.15% )
>>
>> I also fixed a bug in Images.gmk that always wrote the default archive to $JAVA_HOME/lib/server. This fix also makes it possible for a client libjvm to have a default CDS archive.
>
> Ioi Lam has updated the pull request incrementally with two additional commits since the last revision:
>
> - create_cds_archive -> CreateCDSArchive
> - explicitly choose VM variant in create_cds_archive
Build changes look good.
make/Images.gmk line 144:
> 142:
> 143: $$(eval $$(call SetupExecute, $1_$2_gen_cds_archive_jre, \
> 144: WARN := Creating CDS$$($1_$2_DUMP_TYPE) archive for jre image, \
Perhaps include the variant name in the log message here, as is done above for the jdk image?
-------------
Marked as reviewed by erikj (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9869
More information about the build-dev
mailing list