[10] RFR(XS): 8174101: Bootclasspath append should not invalidate CDS archive

Jiangli Zhou jiangli.zhou at oracle.com
Thu Nov 30 18:40:51 UTC 2017


Hi Calvin,

The fix looks ok to me. Could you please add a comment/note above the line130 change:

“In the future we should perform the check based on the content of the mapped archive.”

Thanks,
Jiangli

> On Nov 30, 2017, at 9:49 AM, Calvin Cheung <calvin.cheung at oracle.com> wrote:
> 
> For the regular CDS case (UseAppCDS=0), only boot classes are loaded from the archive. Therefore, we should not invalidate the archive when the bootclasspath is appended.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8174101
> 
> webrev: http://cr.openjdk.java.net/~ccheung/8174101/webrev.00/
> 
> Testing: hs-tier1 and hs-tier2 on linux-x64, solaris-sparc, macosx, windows-x64.
> 
> thanks,
> Calvin



More information about the hotspot-runtime-dev mailing list