RFR (S): JDK-8057910: G1: BOT verification should not pass top
Bengt Rutisson
bengt.rutisson at oracle.com
Tue Sep 9 12:31:59 UTC 2014
Thanks StefanK and Thomas for the quick reviews!
I'll push this directly now to get cleaner tests this night.
Thanks,
Bengt
On 2014-09-09 14:32, Thomas Schatzl wrote:
> Hi,
>
> On Tue, 2014-09-09 at 14:13 +0200, Bengt Rutisson wrote:
>> Hi all,
>>
>> Can I have a couple of reviews for this fix?
>>
>> http://cr.openjdk.java.net/~brutisso/8057910/webrev.00/
>>
>> https://bugs.openjdk.java.net/browse/JDK-8057910
>>
>>
>> After the fix for JDK-8025564 the BOT verification may pass the end for
>> the reserved space. The reason is this code in G1BlockOffsetArray::verify():
>>
>> size_t start_card = _array->index_for(gsp()->bottom());
>> size_t end_card = _array->index_for(gsp()->top());
>>
>> The end_card should not be based on top() but on "top() - 1" to make
>> sure we are inside the heap region and thus inside the reserved space.
>>
>> Testing:
>> - Ad hoc aurora tests simulating nightly testing. The current bug failed
>> a lot in the nightly testing.
>> - JPRT
> looks good.
>
> Thomas
>
>
More information about the hotspot-gc-dev
mailing list