JDK-8213224 : Move code related to GC threads calculation out of AdaptiveSizePolicy
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Affected Version: 12
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2018-10-31
  • Updated: 2018-12-14
  • Resolved: 2018-12-07
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 12
12 b24Fixed
Related Reports
Relates :  
JDK-6593758 - RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
Relates :  
JDK-6858051 - Create GC worker threads dynamically
Description
The following three functions are unrelated to the main functionality of AdaptiveSizePolicy:
calc_default_active_workers()
calc_active_workers()
calc_active_conc_workers()

They were introduced in JDK-6593758 (http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/bca17e38de00). The follow-up JDK-6858051 added a WorkerManager class, which is a better location for these 3 functions.

In addition, G1 does not have AdaptiveSizePolicy, yet currently G1 code needs to include adaptiveSizePolicy.hpp to call 2 of the 3 functions.
Comments
As suggested by reviewer, the following three functions could also be moved to the same (possibly new) class: Abstract_VM_Version::nof_parallel_worker_threads(); Abstract_VM_Version::parallel_worker_threads(); Abstract_VM_Version::calc_parallel_worker_threads();
06-11-2018