RFR: 8302975: Remove redundant mark verification during G1 Full GC

Kim Barrett kbarrett at openjdk.org
Wed Feb 22 07:08:49 UTC 2023


On Tue, 21 Feb 2023 10:52:59 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this remove of some redundant verification code that also opens opportunities for further cleanup in PR#12260. In particular it removes verification of marks during g1 full gc marking: this is checked after marking already, and actually in `G1FullGCMarker::mark_and_push` anyway.
> 
> To make this change small, I did not move the `G1VerifyLiveClosure` somewhere private, and anyway it will be refactored in PR#12260 after this has been pushed again..
> 
> Testing: local compilation
> 
> Thanks,
>   Thomas

Looks good.

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

Marked as reviewed by kbarrett (Reviewer).

PR: https://git.openjdk.org/jdk/pull/12687


More information about the hotspot-gc-dev mailing list