8240511: Shenandoah: parallel safepoint workers count should be Paral…

…lelGCThreads Reviewed-by: rkennke
openjdk · Mar 4, 2020 · b3666b9 · b3666b9
1 parent 8e74ed4
commit b3666b9
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 52 deletions.
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp
@@ -478,9 +478,9 @@ ShenandoahHeap::ShenandoahHeap(ShenandoahCollectorPolicy* policy) :
     _workers->initialize_workers();
   }
 
-  if (ShenandoahParallelSafepointThreads > 1) {
+  if (ParallelGCThreads > 1) {
     _safepoint_workers = new ShenandoahWorkGang("Safepoint Cleanup Thread",
-                                                ShenandoahParallelSafepointThreads,
+                                                ParallelGCThreads,
                       /* are_GC_task_threads */ false,
                  /* are_ConcurrentGC_threads */ false);
     _safepoint_workers->initialize_workers();

diff --git a/src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp b/src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp
@@ -317,9 +317,6 @@
           "Forcefully flush non-empty SATB buffers at this interval. "      \
           "Time is in milliseconds.")                                       \
                                                                             \
-  experimental(uint, ShenandoahParallelSafepointThreads, 4,                 \
-          "Number of parallel threads used for safepoint prolog/epilog")    \
-                                                                            \
   diagnostic(bool, ShenandoahPreclean, true,                                \
           "Do concurrent preclean phase before final mark: process "        \
           "definitely alive references to avoid dealing with them during "  \

diff --git a/test/hotspot/jtreg/gc/shenandoah/options/TestSafepointWorkers.java b/test/hotspot/jtreg/gc/shenandoah/options/TestSafepointWorkers.java