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

Kim Barrett kim.barrett at oracle.com
Wed Apr 15 15:39:35 UTC 2015


On Apr 15, 2015, at 1:12 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi,
> 
> On Wed, 2015-04-15 at 02:57 -0400, 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/
> 
>  looks good.
> 
> Thanks,
>  Thomas

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



More information about the hotspot-gc-dev mailing list