RFR: 8255001: Move G1PeriodicGCTask to its own file [v2]
tschatzl at openjdk.java.net
Fri Nov 27 09:46:09 UTC 2020
On Fri, 27 Nov 2020 09:23:09 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
>> After making the service thread task based it is now possible to move the tasks out of `g1ServiceThread.cpp`. The periodic task is moved to its own files and the initialization and registration with the service thread is done while initializing `G1CollectedHeap`.
>> No functionality is changed for the task, but it has been split up into a header and a source file.
>> tier 1-2 + manual testing setting `G1PeriodicGCInterval` through `jcmd`.
> Stefan Johansson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> - Merge branch 'master' into 8255001-move-periodic-gc-task
> - 8255001: Move G1PeriodicGCTask to its own file
Changes requested by tschatzl (Reviewer).
src/hotspot/share/gc/g1/g1PeriodicGCTask.hpp line 31:
> 29: #include "utilities/globalDefinitions.hpp"
> 30: #include "utilities/ticks.hpp"
Only the g1ServiceThread include seems to be required here. The others can be moved to the cpp file.
More information about the hotspot-gc-dev