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