RFR(8u): JDK-8145442: Add the facility to verify remembered sets for G1

[Jon Masamitsu]

On 01/28/2016 12:37 PM, Poonam Bajaj Parhar wrote:
> Hello Jon,
>
> On 1/28/2016 11:48 AM, Jon Masamitsu wrote:
>> Poonam,
>>
>> Changes look good.
>
> Thanks for the review.
>
>>
>> With VerifyBeforeGC and VerifyAfterGC the gc logs will look
>> a little different?  The rem-set verify part of the log will be separate?
>> Instead of
>>
>> 2.701: [Full GC (Allocation Failure) VerifyBeforeGC:[Verifying 
>> threads heap Roots HeapRegionSets HeapRegions RemSet syms strs zone 
>> dict cldg metaspace chunks hand C-heap code cache ]
>>
>> the rem-set message will be from
>>
>> 4028 if (VerifyRememberedSets) {
>> 4029 if (PrintGCDetails) {
>> 4030 gclog_or_tty->print_cr("[Verifying RemSets before GC]");
>> 4031 }

Should  this code above consider the VerifySilently flag?

Jon
>
> Yes, the output with VerifyRememberedSets will be different from the 
> output with VerifyBeforeGC and VerifyAfterGC flags. Since 
> VerifyRememberedSets is not dependent on VerifyBeforeGC and 
> VerifyAfterGC options, it will be used independently when we don't 
> want to verify everything.
>
> Thanks,
> Poonam
>
>> Jon
>>
>>
>> On 01/27/2016 12:03 PM, Poonam Bajaj Parhar wrote:
>>> Hello,
>>>
>>> Could I get reviews for the backport of the fix for JDK-8145442 
>>> <https://bugs.openjdk.java.net/browse/JDK-8145442> to jdk8u.
>>>
>>> Webrev: http://cr.openjdk.java.net/~poonam/8145442/webrev.8u/
>>> jdk9 changeset: 
>>> http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/3666a5638df2
>>> Testing: JPRT, GCBasher
>>>
>>> Thanks,
>>> Poonam
>>>
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20160128/a4965aa1/attachment.htm>