RFR: 8267894: Skip work for empty regions in G1 Full GC

Ivan Walulya iwalulya at openjdk.java.net
Wed Aug 18 11:30:26 UTC 2021


On Wed, 18 Aug 2021 11:20:13 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

> I think the change looks good in general, but maybe we should go with a "named" state for free regions. Either we change the name for the default value in `G1FullGCHeapRegionAttr` or we add a new state that we set for free regions. What do you think?


AFAIK, changing the default value to "Free" instead of  "Invalid" would suffice, however, need to clarify on how that is handled by `assert(!is_invalid(obj), "not initialized yet");` which indicate some sort of transition from "Invalid" to "initialized".

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

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



More information about the hotspot-gc-dev mailing list