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

Thomas Schatzl tschatzl at openjdk.org
Wed Feb 22 15:49:17 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

This pull request has now been integrated.

Changeset: ee37af47
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/ee37af47cd722bbfb7235bab0af44cb6f7b7863c
Stats:     20 lines in 3 files changed: 0 ins; 20 del; 0 mod

8302975: Remove redundant mark verification during G1 Full GC

Reviewed-by: ayang, kbarrett

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

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


More information about the hotspot-gc-dev mailing list