Integrated: 8271060: Merge G1CollectedHeap::determine_start_concurrent_mark_gc and G1Policy::decide_on_conc_mark_initiation
Thomas Schatzl
tschatzl at openjdk.java.net
Fri Jul 23 09:55:06 UTC 2021
On Thu, 22 Jul 2021 08:46:36 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
> Hi all,
>
> can I have reviews for this merge of two methods that have the same purpose: determine whether this pause should be a concurrent start pause. The reason for this merge is that `G1CollectedHeap::determine_start_concurrent_mark_gc` just calls the other, and
> - is the only caller
> - only adds another condition of the same type that `G1Policy::decide_on_conc_mark_initiation` does at the beginning
> - adds some return value checking and return value
> which all imho fit into `G1Policy::decide_on_conc_mark_initiation` too.
>
> It also reduces `G1CollectedHeap` a bit. The main reason for me to put this in the new location is basically that that additional condition in `G1CollectedHeap::determine_start_concurrent_mark_gc` makes all conditions for that decision be located closer together.
>
> If you think otherwise and this is a bad idea, I will close this PR.
>
> Testing: manual gc/g1 runs
>
> Thanks,
> Thomas
This pull request has now been integrated.
Changeset: ea182b5c
Author: Thomas Schatzl <tschatzl at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/ea182b5c6eba5f7e66b324068d3b42ffb2d29190
Stats: 50 lines in 5 files changed: 20 ins; 20 del; 10 mod
8271060: Merge G1CollectedHeap::determine_start_concurrent_mark_gc and G1Policy::decide_on_conc_mark_initiation
Reviewed-by: ayang, kbarrett
-------------
PR: https://git.openjdk.java.net/jdk/pull/4867
More information about the hotspot-gc-dev
mailing list