RFR: JDK-8262068: Improve G1 Full GC by skipping compaction for regions with high survival ratio [v11]

Hamlin Li mli at openjdk.java.net
Sat Mar 27 06:02:26 UTC 2021


On Fri, 26 Mar 2021 10:51:55 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> I'm not sure, but per code suggests in src/hotspot/share/gc/g1/heapRegionType.hpp, seems closed region should be a pinned as well. Or miss something? Would you mind to clarify this point?
>>     ArchiveMask           = 32,
>>     OpenArchiveTag        = ArchiveMask | PinnedMask,
>>     ClosedArchiveTag      = ArchiveMask | PinnedMask + 1
>
> Just because a `ClosedArchive` region is by default pinned at this time, it does not mean it will be in the future. There are no immediate plans for that, but I can think of some edge cases where it could be useful.
> 
> The correct (and intended) predicate for this check in this context is `is_pinned()`, although at the moment they have the same effect.

I see, thanks for clarifying.

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

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



More information about the hotspot-gc-dev mailing list