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

[Thomas Schatzl]

On Thu, 26 Jan 2023 22:18:04 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

>> No, `!full_gc` means we reach here because concurrent marking is about to start and we want to arm unconditionally then. Never arm for the start of a full gc on the other hand.
>> 
>> Would it be clearer if I renamed `full_gc` to `concurrent_marking_start`?
>
> Yes, I think so. Otherwise, looks good. Thanks again!

please use `concurrent_mark_start` (without the "ing"), or `in_concurrent_start_mark[_pause]`.

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

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