comparison src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp @ 20274:a8137787acfe

8047821: G1 Does not use the save_marks functionality as intended Summary: Rename save_marks to record_top_and_timestamp and remove som unused but related methods Reviewed-by: stefank, ehelin
author mgerdin
date Thu, 26 Jun 2014 10:00:00 +0200
parents b7d24d2bc8be
children 2c6ef90f030a
comparison
equal deleted inserted replaced
20273:9441d22e429a 20274:a8137787acfe
4359 if (retained_region != NULL && 4359 if (retained_region != NULL &&
4360 !retained_region->in_collection_set() && 4360 !retained_region->in_collection_set() &&
4361 !(retained_region->top() == retained_region->end()) && 4361 !(retained_region->top() == retained_region->end()) &&
4362 !retained_region->is_empty() && 4362 !retained_region->is_empty() &&
4363 !retained_region->isHumongous()) { 4363 !retained_region->isHumongous()) {
4364 retained_region->set_saved_mark(); 4364 retained_region->record_top_and_timestamp();
4365 // The retained region was added to the old region set when it was 4365 // The retained region was added to the old region set when it was
4366 // retired. We have to remove it now, since we don't allow regions 4366 // retired. We have to remove it now, since we don't allow regions
4367 // we allocate to in the region sets. We'll re-add it later, when 4367 // we allocate to in the region sets. We'll re-add it later, when
4368 // it's retired again. 4368 // it's retired again.
4369 _old_set.remove(retained_region); 4369 _old_set.remove(retained_region);
6302 true /* do_expand */); 6302 true /* do_expand */);
6303 if (new_alloc_region != NULL) { 6303 if (new_alloc_region != NULL) {
6304 // We really only need to do this for old regions given that we 6304 // We really only need to do this for old regions given that we
6305 // should never scan survivors. But it doesn't hurt to do it 6305 // should never scan survivors. But it doesn't hurt to do it
6306 // for survivors too. 6306 // for survivors too.
6307 new_alloc_region->set_saved_mark(); 6307 new_alloc_region->record_top_and_timestamp();
6308 if (survivor) { 6308 if (survivor) {
6309 new_alloc_region->set_survivor(); 6309 new_alloc_region->set_survivor();
6310 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor); 6310 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor);
6311 } else { 6311 } else {
6312 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Old); 6312 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Old);