RFR: 8069367: assert(_nextMarkBitMap->isMarked((HeapWord*) obj)) failed

[Kim Barrett]

On Apr 15, 2015, at 4:38 AM, Bengt Rutisson <bengt.rutisson at oracle.com> wrote:
> 
> 
> On 2015-04-15 08:57, Kim Barrett wrote:
>> Another round.
>> 
>> Replaced add/remove_candidate with setter for candidate set.  Replaced
>> add/remove_humongous_reclaim_candidate modifiers with setter for
>> G1CollectedHeap.  These changes allow some code simplifications in the
>> candidate selection.
>> 
>> Share common arguments for initializing _in_cset_fast_test and
>> _humongous_reclaim_candidates, rather than recomputing for each.
>> 
>> Improved various comments based on review feedback.
>>  CR:
>> https://bugs.openjdk.java.net/browse/JDK-8069367
>> 
>> Webrev:
>> http://cr.openjdk.java.net/~kbarrett/8069367/webrev.05/
>> 
>> Incremental webrev:
>> http://cr.openjdk.java.net/~kbarrett/8069367/webrev.05.incr/
> 
> Looks good to me.
> 
> In particular the updated comments look fine. Thanks for fixing this!
> 
> Bengt

Thanks for your review, and for your many suggested improvements.