8190711: Assert in G1MMUTracker due to concurrent modification
Reviewed-by: tschatzl, rehn, sangheki
--- a/src/hotspot/share/gc/g1/g1MMUTracker.cpp Thu Nov 09 12:12:32 2017 +0530
+++ b/src/hotspot/share/gc/g1/g1MMUTracker.cpp Thu Nov 09 10:11:19 2017 +0100
@@ -112,18 +112,7 @@
}
}
-// basically the _internal call does not remove expired entries
-// this is for trying things out in the future and a couple
-// of other places (debugging)
-
double G1MMUTrackerQueue::when_sec(double current_time, double pause_time) {
- remove_expired_entries(current_time);
-
- return when_internal(current_time, pause_time);
-}
-
-double G1MMUTrackerQueue::when_internal(double current_time,
- double pause_time) {
// if the pause is over the maximum, just assume that it's the maximum
double adjusted_pause_time =
(pause_time > max_gc_time()) ? max_gc_time() : pause_time;
--- a/src/hotspot/share/gc/g1/g1MMUTracker.hpp Thu Nov 09 12:12:32 2017 +0530
+++ b/src/hotspot/share/gc/g1/g1MMUTracker.hpp Thu Nov 09 10:11:19 2017 +0100
@@ -134,8 +134,6 @@
void remove_expired_entries(double current_time);
double calculate_gc_time(double current_time);
- double when_internal(double current_time, double pause_time);
-
public:
G1MMUTrackerQueue(double time_slice, double max_gc_time);