RFR(s) 8154457: runtime/SharedArchiveFile/SharedStrings Shared	string table stats missing
    Lois Foltan 
    lois.foltan at oracle.com
       
    Wed Apr 20 11:51:45 UTC 2016
    
    
  
On 4/19/2016 9: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.
Hi Jiangli,
I think this looks good. One very minor comment:
src/share/vm/classfile/stringTable.cpp
"Current setting:" should probably be "Current settings:"
I don't need to see another webrev.
Thanks,
Lois
>
> Thanks,
> Jiangli
    
    
More information about the hotspot-runtime-dev
mailing list