diff --git a/src/hotspot/share/gc/g1/g1CollectedHeap.cpp b/src/hotspot/share/gc/g1/g1CollectedHeap.cpp index 2368b5b4fc432..13326dd1d2281 100644 --- a/src/hotspot/share/gc/g1/g1CollectedHeap.cpp +++ b/src/hotspot/share/gc/g1/g1CollectedHeap.cpp @@ -2669,8 +2669,7 @@ void G1CollectedHeap::do_collection_pause_at_safepoint_helper(size_t allocation_ } void G1CollectedHeap::complete_cleaning(bool class_unloading_occurred) { - uint num_workers = workers()->active_workers(); - G1ParallelCleaningTask unlink_task(num_workers, class_unloading_occurred); + G1ParallelCleaningTask unlink_task(class_unloading_occurred); workers()->run_task(&unlink_task); } diff --git a/src/hotspot/share/gc/g1/g1ParallelCleaning.cpp b/src/hotspot/share/gc/g1/g1ParallelCleaning.cpp index 884a2b2ca55d9..8d5e2a3239cb6 100644 --- a/src/hotspot/share/gc/g1/g1ParallelCleaning.cpp +++ b/src/hotspot/share/gc/g1/g1ParallelCleaning.cpp @@ -50,11 +50,10 @@ void JVMCICleaningTask::work(bool unloading_occurred) { } #endif // INCLUDE_JVMCI -G1ParallelCleaningTask::G1ParallelCleaningTask(uint num_workers, - bool unloading_occurred) : +G1ParallelCleaningTask::G1ParallelCleaningTask(bool unloading_occurred) : WorkerTask("G1 Parallel Cleaning"), _unloading_occurred(unloading_occurred), - _code_cache_task(num_workers, unloading_occurred), + _code_cache_task(unloading_occurred), JVMCI_ONLY(_jvmci_cleaning_task() COMMA) _klass_cleaning_task() { } diff --git a/src/hotspot/share/gc/g1/g1ParallelCleaning.hpp b/src/hotspot/share/gc/g1/g1ParallelCleaning.hpp index df9b1dddf6c29..1a4e460182797 100644 --- a/src/hotspot/share/gc/g1/g1ParallelCleaning.hpp +++ b/src/hotspot/share/gc/g1/g1ParallelCleaning.hpp @@ -54,8 +54,7 @@ class G1ParallelCleaningTask : public WorkerTask { public: // The constructor is run in the VMThread. - G1ParallelCleaningTask(uint num_workers, - bool unloading_occurred); + G1ParallelCleaningTask(bool unloading_occurred); void work(uint worker_id); }; diff --git a/src/hotspot/share/gc/shared/parallelCleaning.cpp b/src/hotspot/share/gc/shared/parallelCleaning.cpp index d2f69bfa679e7..e302085d0cc71 100644 --- a/src/hotspot/share/gc/shared/parallelCleaning.cpp +++ b/src/hotspot/share/gc/shared/parallelCleaning.cpp @@ -30,9 +30,8 @@ #include "oops/klass.inline.hpp" #include "runtime/atomicAccess.hpp" -CodeCacheUnloadingTask::CodeCacheUnloadingTask(uint num_workers, bool unloading_occurred) : +CodeCacheUnloadingTask::CodeCacheUnloadingTask(bool unloading_occurred) : _unloading_occurred(unloading_occurred), - _num_workers(num_workers), _first_nmethod(nullptr), _claimed_nmethod(nullptr) { // Get first alive nmethod diff --git a/src/hotspot/share/gc/shared/parallelCleaning.hpp b/src/hotspot/share/gc/shared/parallelCleaning.hpp index b47bf92e2ac2f..e1b07f5778de8 100644 --- a/src/hotspot/share/gc/shared/parallelCleaning.hpp +++ b/src/hotspot/share/gc/shared/parallelCleaning.hpp @@ -33,14 +33,13 @@ class CodeCacheUnloadingTask { const bool _unloading_occurred; - const uint _num_workers; // Variables used to claim nmethods. nmethod* _first_nmethod; nmethod* volatile _claimed_nmethod; public: - CodeCacheUnloadingTask(uint num_workers, bool unloading_occurred); + CodeCacheUnloadingTask(bool unloading_occurred); ~CodeCacheUnloadingTask(); private: diff --git a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp index e6d41237c98a3..b2fd32d2fd0b9 100644 --- a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp +++ b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp @@ -2259,8 +2259,7 @@ void ShenandoahHeap::stw_unload_classes(bool full_gc) { // Clean JVMCI metadata handles. JVMCI_ONLY(JVMCI::do_unloading(unloading_occurred)); - uint num_workers = _workers->active_workers(); - ShenandoahClassUnloadingTask unlink_task(phase, num_workers, unloading_occurred); + ShenandoahClassUnloadingTask unlink_task(phase, unloading_occurred); _workers->run_task(&unlink_task); } // Release unloaded nmethods's memory. diff --git a/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.cpp b/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.cpp index 99c739026cc7d..6c82f970606b9 100644 --- a/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.cpp +++ b/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.cpp @@ -31,12 +31,11 @@ #include "runtime/safepoint.hpp" ShenandoahClassUnloadingTask::ShenandoahClassUnloadingTask(ShenandoahPhaseTimings::Phase phase, - uint num_workers, bool unloading_occurred) : WorkerTask("Shenandoah Class Unloading"), _phase(phase), _unloading_occurred(unloading_occurred), - _code_cache_task(num_workers, unloading_occurred), + _code_cache_task(unloading_occurred), _klass_cleaning_task() { assert(SafepointSynchronize::is_at_safepoint(), "Must be at a safepoint"); } diff --git a/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.hpp b/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.hpp index b4c4fb94a5aa3..c68a4fde4ce4d 100644 --- a/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.hpp +++ b/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.hpp @@ -59,7 +59,6 @@ class ShenandoahClassUnloadingTask : public WorkerTask { KlassCleaningTask _klass_cleaning_task; public: ShenandoahClassUnloadingTask(ShenandoahPhaseTimings::Phase phase, - uint num_workers, bool unloading_occurred); void work(uint worker_id);