RFR: 8392760: Improve liveness/remembered set verification for G1 [v2]

Thomas Schatzl tschatzl at openjdk.org
Wed Feb 22 09:43:46 UTC 2023


On Wed, 22 Feb 2023 06:52:57 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - Refactoring after 8302975
>>  - initial attempts, nothing done
>>    
>>    further cleanup
>>    
>>    more cleanup
>>    
>>    factor out changes that move the verifylivenessclosure
>>    
>>    fix compilation
>>    
>>    kbarrett review, indentation
>>    
>>    Minor removal of unnecessary code
>>    
>>    initial version
>>    
>>    more cleanup
>>    
>>    More improvements
>>    
>>    More removal of stuff
>>    
>>    More cleanup
>>    
>>    More^2 cleanup
>>    
>>    Remove some changes
>>    
>>    More cleanups
>>    
>>    fix compilation
>>    
>>    Cleanup
>>  - removal
>
> src/hotspot/share/gc/g1/heapRegion.cpp line 517:
> 
>> 515:       LogStream ls(log.error());
>> 516: 
>> 517:       MutexLocker x(G1RareEvent_lock, Mutex::_no_safepoint_check_flag);
> 
> [pre-existing] It's not clear to me what this lock is protecting.  Similarly for the one in `verify_remset`
> (in an unmodified part of it).

Having this lock avoids the log messages for a single failures (these are multiple lines each) to not mix. Otherwise it would be very hard to analyze the failures. Does that make sense?

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

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


More information about the hotspot-gc-dev mailing list