On Tue, 15 Sep 2020 13:51:22 GMT, Ivan Walulya <iwalulya@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
This pull request has now been integrated. Changeset: 86a16400 Author: Ivan Walulya <iwalulya@openjdk.org> Committer: Thomas Schatzl <tschatzl@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/86a16400 Stats: 51 lines in 4 files changed: 11 ins; 37 del; 3 mod 8244505: G1 pause time ratio calculation does not consider Remark/Cleanup pauses Reviewed-by: tschatzl, sjohanss ------------- PR: https://git.openjdk.java.net/jdk/pull/183