On Fri, 17 Nov 2023 05:38:28 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> src/hotspot/share/prims/jvmtiEventController.cpp line 372:
>> 
>>> 370:     return;  // EnterInterpOnlyModeClosure will be executed right 
>>> after mount.
>>> 371:   }
>>> 372:   ThreadsListHandle tlh(current);
>> 
>> Why was this added?
>
> This is explained in the PR description.
> Do you think, a just comment is needed or this has to be separated from this 
> fix?

I see the PR comment, but I don't really understand it. Is this to force some 
sort of early initialization to avoid a race later on? It just seems odd to 
create the tlh, but never use it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16686#discussion_r1396735115

Reply via email to