RFR: 8367993: G1: Speed up ConcurrentMark initialization [v3]

[Leo Korinth]

On Thu, 8 Jan 2026 13:29:25 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

>> Leo Korinth has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 564 commits:
>> 
>>  - Merge branch '8373253' into 8367993
>>  - Merge branch 'master' into _8373253
>>  - Merge branch 'master' into _8367993
>>  - 8366058: Outdated comment in WinCAPISeedGenerator
>>    
>>    Reviewed-by: mullan
>>  - 8357258: x86: Improve receiver type profiling reliability
>>    
>>    Reviewed-by: kvn, vlivanov
>>  - 8373704: Improve "SocketException: Protocol family unavailable" message
>>    
>>    Reviewed-by: lucy, jpai
>>  - 8373722: [TESTBUG] compiler/vectorapi/TestVectorOperationsWithPartialSize.java fails intermittently
>>    
>>    Reviewed-by: jiefu, jbhateja, erfang, qamai
>>  - 8343809: Add requires tag to mark tests that are incompatible with exploded image
>>    
>>    Reviewed-by: alanb, dholmes
>>  - 8374465: Spurious dot in documentation for JVMTI ClassLoad
>>    
>>    Reviewed-by: kbarrett
>>  - 8374317: Change GCM IV size to 12 bytes when encrypting/decrypting TLS session ticket
>>    
>>    Reviewed-by: djelinski, mpowers, ascarpino
>>  - ... and 554 more: https://git.openjdk.org/jdk/compare/2aa8aa4b...28ccbb68
>
> src/hotspot/share/gc/g1/g1CollectedHeap.cpp line 1637:
> 
>> 1635: 
>> 1636: bool G1CollectedHeap::concurrent_mark_is_terminating() const {
>> 1637:   assert(_cm != nullptr, "thread must exist in order to check if mark is terminating");
> 
> I think it would make sense to add `&& _cm->is_fully_initialized()` to really make sure the thread has been created.

Yes. I will make it two lines instead, easier to see what assertion that fails.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28723#discussion_r2676599920