On Thu, 19 Oct 2023 01:22:46 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Jorn Vernee has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   add s390 support
>
> src/hotspot/share/gc/shared/gcLocker.cpp line 139:
> 
>> 137:     // has called `jni_unlock`, but not yet finished the call, e.g. 
>> initiating
>> 138:     // a GCCause::_gc_locker GC.
>> 139:     log_debug_jni("Blocked from entering critical section while waiting 
>> on GC.");
> 
> Seems unrelated to current MR - leftover debugging code?

Yes, good catch. Leftover from when this patch used GCLocker (which turned out 
to be not needed in the end)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16201#discussion_r1364783686

Reply via email to