RFR: 8236176: Parallel GC SplitInfo comment should be updated for shadow regions [v2]
Ivan Walulya
iwalulya at openjdk.java.net
Mon Aug 23 16:07:50 UTC 2021
On Mon, 23 Aug 2021 11:13:22 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Ivan Walulya has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Thomas review
>
> src/hotspot/share/gc/parallel/psParallelCompact.hpp line 964:
>
>> 962: // region that can be put on the ready list. The regions are atomically added
>> 963: // and removed from the ready list.
>> 964: //
>
> This commentactually refers to the sentence: "// An alternate strategy is being investigated for this deferral of updating." above.
>
> This sentence should be removed as this offers no information about the current algorithm. Fwiw I do not know that actually anyone is working on that, and if so, a CR in JIRA would be more appropriate. I could not find anything in JIRA about work on this, and given that comment is pretty old, this is just confusing.
Removed!
-------------
PR: https://git.openjdk.java.net/jdk/pull/5195
More information about the hotspot-gc-dev
mailing list