Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8270014: Add scoped objects for g1 young gc verification and young gc internal timing #4768

Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
@@ -2948,8 +2948,8 @@ class G1YoungGCVerifierMark : public StackObj {

class G1YoungGCNotifyPauseMark : public StackObj {
public:
G1YoungGCNotifyPauseMark() { G1CollectedHeap::heap()->policy()->note_young_gc_pause_start(); }
~G1YoungGCNotifyPauseMark() { G1CollectedHeap::heap()->policy()->note_young_gc_pause_end(); }
G1YoungGCNotifyPauseMark() { G1CollectedHeap::heap()->policy()->record_young_gc_pause_start(); }
~G1YoungGCNotifyPauseMark() { G1CollectedHeap::heap()->policy()->record_young_gc_pause_end(); }
};

G1HeapPrinterMark::G1HeapPrinterMark(G1CollectedHeap* g1h) : _g1h(g1h), _heap_transition(g1h) {
@@ -113,7 +113,7 @@ void G1Policy::init(G1CollectedHeap* g1h, G1CollectionSet* collection_set) {
_collection_set->start_incremental_building();
}

void G1Policy::note_young_gc_pause_start() {
void G1Policy::record_young_gc_pause_start() {
phase_times()->record_gc_pause_start();
}

@@ -891,7 +891,7 @@ void G1Policy::report_ihop_statistics() {
_ihop_control->print();
}

void G1Policy::note_young_gc_pause_end() {
void G1Policy::record_young_gc_pause_end() {
phase_times()->record_gc_pause_end();
phase_times()->print();
}
@@ -1490,7 +1490,7 @@ void G1Policy::update_survival_estimates_for_next_collection() {
}

void G1Policy::transfer_survivors_to_cset(const G1SurvivorRegions* survivors) {
note_start_adding_survivor_regions();
start_adding_survivor_regions();

HeapRegion* last = NULL;
for (GrowableArrayIterator<HeapRegion*> it = survivors->regions()->begin();
@@ -1506,7 +1506,7 @@ void G1Policy::transfer_survivors_to_cset(const G1SurvivorRegions* survivors) {

last = curr;
}
note_stop_adding_survivor_regions();
stop_adding_survivor_regions();

// Don't clear the survivor list handles until the start of
// the next evacuation pause - we need it in order to re-tag
@@ -302,8 +302,8 @@ class G1Policy: public CHeapObj<mtGC> {
void init(G1CollectedHeap* g1h, G1CollectionSet* collection_set);

// Record the start and end of the young gc pause.
void note_young_gc_pause_start();
void note_young_gc_pause_end();
void record_young_gc_pause_start();
void record_young_gc_pause_end();

bool need_to_start_conc_mark(const char* source, size_t alloc_word_size = 0);

@@ -427,11 +427,11 @@ class G1Policy: public CHeapObj<mtGC> {
return _max_survivor_regions;
}

void note_start_adding_survivor_regions() {
void start_adding_survivor_regions() {
_survivor_surv_rate_group->start_adding_regions();
}

void note_stop_adding_survivor_regions() {
void stop_adding_survivor_regions() {
_survivor_surv_rate_group->stop_adding_regions();
}