RFR: 8300915: G1: incomplete SATB because nmethod entry barriers don't get armed [v2]

[Richard Reingruber]

On Wed, 25 Jan 2023 16:17:17 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Richard Reingruber has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve assert
>>   
>>   Co-authored-by: Thomas Schatzl <59967451+tschatzl at users.noreply.github.com>
>
> src/hotspot/share/gc/g1/g1ConcurrentMark.cpp line 796:
> 
>> 794: 
>> 795:   CodeCache::on_gc_marking_cycle_start();
>> 796:   CodeCache::arm_all_nmethods();
> 
> Maybe factor this out into a method (`start_codecache_marking_cycle`) and call it above.

Like this right?

```c++
void G1CollectedHeap::start_codecache_marking_cycle() {
  CodeCache::on_gc_marking_cycle_start();
  CodeCache::arm_all_nmethods();
}

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

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