RFR: 8274466 G1: simplify G1 collector states

Stefan Johansson sjohanss at openjdk.java.net
Wed Sep 29 12:41:39 UTC 2021


On Wed, 29 Sep 2021 01:09:41 GMT, Hamlin Li <mli at openjdk.org> wrote:

> This is a minor improvement which remove the redundant G1 Collector states usages.

src/hotspot/share/gc/g1/g1Policy.cpp line 604:

> 602:   bool result = false;
> 603:   if (marking_request_bytes > marking_initiating_used_threshold) {
> 604:     result = collector_state()->in_young_only_phase();

To me this looks like a change in behavior. The check you have removed is there to prevent requesting a new concurrent cycle when in the "Prepare Mixed" collections (the young GC preceding mixed collections).

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

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



More information about the hotspot-gc-dev mailing list