RFR(XXS) 8249624: update appcds/sharedStrings/IncompatibleOptions test in view of 8081416 closed as WNF

[Yumin Qi]

Hi Ioi,

   I have updated the words as your suggestion, also more precisely for 
the max heap size for compressed oop is around 31G, which is calculated 
by max_heap_for_compressed_oops().

   updated on same webrev.

$J6/bin/java -Xshare:on 
-XX:SharedArchiveFile=$J6/lib/server/classes.jsa  -Xmx31G -version
java version "16-internal" 2021-03-16
Java(TM) SE Runtime Environment (slowdebug build 
16-internal+0-adhoc.minqi.open)
Java HotSpot(TM) 64-Bit Server VM (slowdebug build 
16-internal+0-adhoc.minqi.open, mixed mode, sharing)

$J6/bin/java -Xshare:on 
-XX:SharedArchiveFile=$J6/lib/server/classes.jsa  -Xmx32G -version
An error has occurred while processing the shared archive file.
Unable to use shared archive.
The saved state of UseCompressedOops and UseCompressedClassPointers is 
different from runtime, CDS will be disabled.
Error occurred during initialization of VM
Unable to use shared archive.


Thanks

Yumin

On 7/22/20 2:06 PM, Ioi Lam wrote:
> Hi Yumin,
>
> Just small nits on the comments:
>
> // UseCompressedOops default is turned on when heap is under 32G but 
> will be
>
> -> UseCompressedOops is turned on by default ....
>
> // turned off when heap is greater than 32G. This leads inconsistency
>
> -> This leads to inconsistency ...
>
> // of UseCompressedOops at dump time and runtime.
>
>
> Thanks
> - Ioi
>
> On 7/22/20 1:47 PM, Yumin Qi wrote:
>> Hi, Please review this tiny change on comment:
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8249624
>>
>> webrev: http://cr.openjdk.java.net/~minqi/2020/8249624/webrev/
>>
>>
>> Note 8081416 already marked as fixed (thanks Ioi), please read the 
>> comment on https://bugs.openjdk.java.net/browse/JDK-8081416
>>
>> With CDS can be done with UseCompressedOops disabled, the test 
>> already has correct result.
>>
>>
>> Thanks
>>
>> Yumin
>>
>>
>