RFR (S): 8208061: runtime/LoadClass/TestResize.java fails with "Load factor too high" when running in CDS mode

[Gerard Ziemski]

hi Jiangli,

Just to make sure I follow your fix, the change to src/hotspot/share/classfile/classLoaderData.cpp is the only one that is actually required and the change to src/hotspot/share/classfile/systemDictionary.cpp is just about making it explicit for the reader?

Looks good.


cheers

> On Aug 23, 2018, at 7:40 PM, Jiangli Zhou <jiangli.zhou at oracle.com> wrote:
> 
> Please review the bug fix for JDK-8208061. The runtime/LoadClass/TestResize.java test failed with "Load factor too high" when running with CDS enabled at runtime. Dictionary resizing was disable when UseSharedSpaces was true, which caused the failure.
> 
> The fix is to allow resizing for all system dictionaries except for the shared dictionary at runtime. The shared dictionary data are from mapped CDS archive and does not change at runtime, so resizing is not required. The change also adds 'resizable' information to the log output in Dictionary::print_on().
> 
> webrev: http://cr.openjdk.java.net/~jiangli/8208061/webrev.00
> bug: https://bugs.openjdk.java.net/browse/JDK-8208061
> 
> Verified with TestResize.java running locally with CDS enabled. Tier1 - tier5 tests are in progress.
> 
> Thanks,
> 
> Jiangli
>