RFR (XS): 8054808: Bitmap verification sometimes fails after Full GC aborts concurrent mark

[Bengt Rutisson]

Hi Thomas,

Looks good. Thanks for fixing this.

Bengt



On 2014-08-28 10:34, Thomas Schatzl wrote:
> Hi Bengt and Jon,
>
>    thanks for the reviews :)
>
> On Thu, 2014-08-28 at 10:02 +0200, Bengt Rutisson wrote:
>> Hi Thomas,
>>
>> (Had an offline discussion with Thomas about this. Replying here for the
>> record.)
>>
>> The proposed fix works fine but there is a slightly simpler solution.
>> For all HeapRegions we store the original end value in a variable that
>> does not change. We could use that for this fix instead of having to
>> calculate the end value.
>>
>> Something like:
>>
>> HeapWord* end = r->orig_end();
>> return _bitmap->getNextMarkedWordAddress(r->bottom(), end) != end;
>>
> New webrev implementing this at
> http://cr.openjdk.java.net/~tschatzl/8054808/webrev.1/
>
> Thanks,
>    Thomas
>
>