On Fri, 5 May 2023 16:38:13 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Several Handlers class use monitors to synchronize when formatting / 
>> publishing LogRecords.
>> When logging within a VirtualThread, holding this monitor can cause the 
>> carrier thread to be pinned.
>> Handlers could use jdk.internal.misc.InternalLock, in a similar way to some 
>> java.io classes (such as PrintStream) to avoid pinning the carrier thread.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Missed changes to SocketHandler.java

Hmm... You're right. Let's use locks consistently then.

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

PR Comment: https://git.openjdk.org/jdk/pull/13832#issuecomment-1536627949

Reply via email to