RFR: 8300915: G1: incomplete SATB because nmethod entry barriers don't get armed [v2]
Thomas Schatzl
tschatzl at openjdk.org
Wed Jan 25 20:19:52 UTC 2023
On Wed, 25 Jan 2023 17:39:47 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> 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();
> }
Exactly. Also I meant "... to call that method here and above"; i.e. I asked to factor out because this is done twice.
-------------
PR: https://git.openjdk.org/jdk/pull/12194
More information about the hotspot-gc-dev
mailing list