--- a/src/hotspot/share/gc/g1/g1CollectedHeap.cpp Sun Oct 27 10:14:40 2019 +0100
+++ b/src/hotspot/share/gc/g1/g1CollectedHeap.cpp Tue Oct 29 08:56:10 2019 +0100
@@ -1630,7 +1630,6 @@
}
jint G1CollectedHeap::initialize() {
- os::enable_vtime();
// Necessary to satisfy locking discipline assertions.
@@ -3614,6 +3613,7 @@
p->record_or_add_time_secs(termination_phase, worker_id, cl.term_time());
p->record_or_add_thread_work_item(termination_phase, worker_id, cl.term_attempts());
}
+ assert(pss->trim_ticks().seconds() == 0.0, "Unexpected partial trimming during evacuation");
}
virtual void start_work(uint worker_id) { }
@@ -3655,22 +3655,14 @@
class G1EvacuateRegionsTask : public G1EvacuateRegionsBaseTask {
G1RootProcessor* _root_processor;
- void verify_trim_ticks(G1ParScanThreadState* pss, const char* location) {
- assert(pss->trim_ticks().seconds() == 0.0, "Unexpected partial trimming during evacuation at %s %.3lf " JLONG_FORMAT, location, pss->trim_ticks().seconds(), pss->trim_ticks().value());
- }
-
void scan_roots(G1ParScanThreadState* pss, uint worker_id) {
_root_processor->evacuate_roots(pss, worker_id);
- verify_trim_ticks(pss, "roots");
_g1h->rem_set()->scan_heap_roots(pss, worker_id, G1GCPhaseTimes::ScanHR, G1GCPhaseTimes::ObjCopy);
- verify_trim_ticks(pss, "heap roots");
_g1h->rem_set()->scan_collection_set_regions(pss, worker_id, G1GCPhaseTimes::ScanHR, G1GCPhaseTimes::CodeRoots, G1GCPhaseTimes::ObjCopy);
- verify_trim_ticks(pss, "scan cset");
}
void evacuate_live_objects(G1ParScanThreadState* pss, uint worker_id) {
G1EvacuateRegionsBaseTask::evacuate_live_objects(pss, worker_id, G1GCPhaseTimes::ObjCopy, G1GCPhaseTimes::Termination);
- verify_trim_ticks(pss, "evac live");
}
void start_work(uint worker_id) {
@@ -4106,7 +4098,7 @@
_cl.complete_work();
G1Policy* policy = G1CollectedHeap::heap()->policy();
- policy->record_max_rs_length(_rs_length);
+ policy->record_rs_length(_rs_length);
policy->cset_regions_freed();
}
public: