8190711: Assert in G1MMUTracker due to concurrent modification
authorsjohanss
Thu, 09 Nov 2017 10:11:19 +0100
changeset 47817 5d15fd7e9bb1
parent 47816 ac0af7750da9
child 47818 2f6ab27efb60
8190711: Assert in G1MMUTracker due to concurrent modification Reviewed-by: tschatzl, rehn, sangheki
src/hotspot/share/gc/g1/g1MMUTracker.cpp
src/hotspot/share/gc/g1/g1MMUTracker.hpp
--- 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);