RFR(s) 8154457: runtime/SharedArchiveFile/SharedStrings Shared string table stats missing

Mikhailo Seledtsov mikhailo.seledtsov at oracle.com
Wed Apr 20 14:12:07 UTC 2016


Hi Jiangli,

  The test changes look good, except for use of System.exit(). I 
recommend throwing exception instead. No need to repost review for this 
change.

Thank you,
Misha

On 4/19/2016 6:26 PM, Jiangli Zhou wrote:
> Please review the following change for JDK-8154457 <https://bugs.openjdk.java.net/browse/JDK-8154457>.
>
>    http://cr.openjdk.java.net/~jiangli/8154457/webrev.00/
>
> Added more information when print out message reporting disabling the shared strings at CDS dump time. The shared strings support requires UseG1GC, UseCompressedOops  and UseCompressedClassPointers all to be true. There are cases where the UseCompressedOops and UseCompressedClassPointers options are disabled due to other VM settings. For example, the UseCompressedOops  and UseCompressedClassPointers are turned off if the VM is running in interpreter only mode, or the if MaxHeapSize is too large. The printed message now includes the value of those three options.
>
> Also, handle the cases when shared strings are disabled in SharedStrings.java test.
>
> Thanks,
> Jiangli



More information about the hotspot-runtime-dev mailing list