I just ran across some code in g1CollectedHeap::collection_set_iterate_from() that looks a little strange. It's been there since the first revision 342. if (cl->doHeapRegion(cur) && false) { cl->incomplete(); return; } Is there any reason why this isn't just cl->doHeapRegion(cur); joe