RFR: 8367993: G1: Speed up ConcurrentMark initialization [v3]
Leo Korinth
lkorinth at openjdk.org
Thu Jan 22 12:33:01 UTC 2026
On Mon, 12 Jan 2026 13:32:35 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> src/hotspot/share/gc/g1/g1Policy.cpp line 744:
>>
>>> 742: if (!_g1h->concurrent_mark()->is_fully_initialized()) {
>>> 743: return false;
>>> 744: }
>>
>> Is this needed? The `in_progress()` check below makes sure to only check the cm_thread when fully initialized.
>
> And this can never return true during initialization as no GC can have happened so that we end up in the Prepare Mixed GC
I removed it, and it seems to work.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28723#discussion_r2716695044
More information about the hotspot-gc-dev
mailing list