RFR (S): 8051973: Eager reclaim leaves marks of marked but reclaimed objects on the next bitmap
Jon Masamitsu
jon.masamitsu at oracle.com
Wed Jul 30 14:28:16 UTC 2014
On 07/30/2014 04:29 AM, Thomas Schatzl wrote:
> Hi Jon,
>
> thanks for the review.
>
> On Tue, 2014-07-29 at 13:50 -0700, Jon Masamitsu wrote:
>> On 7/29/2014 3:40 AM, Thomas Schatzl wrote:
>>> Hi all,
>>>
>>> can I have reviews for the following small change?
>>>
>>> JDK-8027959 implemented eager reclaim of humongous objects, which
>>> potentially left stray set marks on the mark bitmap after reclaiming a
>>> region.
>>>
>>> This change fixes this.
>>>
>>> CR:
>>> https://bugs.openjdk.java.net/browse/JDK-8051973
>>> Webrev:
>>> http://cr.openjdk.java.net/~tschatzl/8051973/webrev/
>> The changes look good.
>>
>> You left the uses r->bottom() so you would not
>> have to cast obj to HeapWord*?
> Yes. If you have objections to that I can change that back. The code is
> easier to read for me if there is not that much casting.
Don't change it back. I like the introduction of "obj". I just
wasn't sure why it was not used in other places to replace
r->bottom(). To avoid casts (a good thing) is sufficient
reason.
>> Do you want to specify a region size in the test? Current
>> policy would have regions sizes such that
>>
>> 48 public static final int M = 1024*1024;
>>
>> gives you humongous allocations but policy could change, yes?
> Done.
Thanks.
>
> See
> http://cr.openjdk.java.net/~tschatzl/8051973/webrev.0_to_1
Looks good.
Jon
> Full webrev:
> http://cr.openjdk.java.net/~tschatzl/8051973/webrev.1
>
> Thomas
>
>
More information about the hotspot-gc-dev
mailing list