RFR: 8349688: G1: Wrong initial optional region index when selecting candidates from retained regions [v2]

Ivan Walulya iwalulya at openjdk.org
Tue Feb 18 19:13:58 UTC 2025


On Tue, 18 Feb 2025 10:47:05 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Ivan Walulya has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'upstream/master' into JDK-8349688-OptionalRegions
>>  - set_index_in_opt_cset correctly
>
> src/hotspot/share/gc/g1/g1CollectionSet.cpp line 486:
> 
>> 484: void G1CollectionSet::select_candidates_from_retained(double time_remaining_ms) {
>> 485:   uint num_initial_regions = 0;
>> 486:   uint prev_num_optional_regions = _optional_groups.num_regions();
> 
> It would be great if the code in `G1CollectionSet::select_candidates_from_marking` would check that the number of optional regions is zero as well when initializing its `num_optional_regions`.

Added, running it through testing.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23568#discussion_r1960061849


More information about the hotspot-gc-dev mailing list