RFR(S) 8153843: G1CardLiveDataHelper incorrectly sets next_live_bytes on dead humongous regions

[Derek White]

On 4/12/16 12:40 PM, Derek White wrote:
> On 4/12/16 8:57 AM, Mikael Gerdin wrote:
>> Hi Derek, Tom
>>
>> Are you planning to re-review webrev.1 or can I push this?
>>
>> http://cr.openjdk.java.net/~mgerdin/8153843/webrev.1/
>>
>> /Mikael
>
> Hi Mikael,
>
> My only question is about the issue Thomas pointed out and you fixed:
> g1CollectedHeap.cpp, Line 2946.
>
> I am surprised that "Dead objects cannot be eager reclaim candidates" 
> -  I thought they were the perfect eager reclaim candidates. Should it 
> be that we simply declare victory and return true at line 2951 
> (bypassing the unsafe stuff)?

OK, answering my own question :-)
  - Derek, please read the big block comment. If the object is dead we 
can't prove all of the constraints listed are satisfied.

But it might be a good idea to tag humongous regions that don't contain 
references at humongous object allocation time. Then we should be able 
to do eager reclaim without looking at the Klass.

reviewed.

  - Derek
>
> Otherwise everything looks fine to me.
>
> Thanks,
>
>  - Derek

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