RFR: 8244505: G1 pause time ratio calculation does not consider Remark/Cleanup pauses [v4]

Stefan Johansson sjohanss at openjdk.java.net
Thu Sep 17 07:38:25 UTC 2020


On Wed, 16 Sep 2020 13:53:42 GMT, Ivan Walulya <iwalulya at openjdk.org> wrote:

>> The computation for long and short term pause time ratios does not include Remark/Cleanup pause times. Isolate updates
>> to _prev_collection_pause_end_ms from update_recent_gc_times(), then have all gc pauses (including Remark/Cleanup)
>> recorded by update_recent_gc_times(). Patch also includes trivial clean ups to g1Policy.* files. Testing: Tier 1 - 7
>> Performance testing did not show any significant changes in performance
>
> Ivan Walulya has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Minor update to change location of method comment

Looks good.

-------------

Marked as reviewed by sjohanss (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/183



More information about the hotspot-gc-dev mailing list