[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