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