On Mon, 3 Mar 2025 15:17:27 GMT, Albert Mingkun Yang <ay...@openjdk.org> wrote:
>> Thomas Schatzl has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> * fix comment (trailing whitespace)
>> * another assert when snapshotting at a safepoint.
>
> src/hotspot/share/gc/g1/g1YoungGCPostEvacuateTasks.cpp line 857:
>
>> 855: // We do not expect too many non-Java threads compared to Java
>> threads, so just
>> 856: // let one worker claim that work.
>> 857: if (!_non_java_threads_claim &&
>> !Atomic::cmpxchg(&_non_java_threads_claim, false, true,
>> memory_order_relaxed)) {
>
> Do non-java threads have card-table-base?
This code should not be necessary (any more). Will remove.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23739#discussion_r1977853483
