RFR: 8308614: Enabling JVMTI ClassLoad event slows down vthread creation by factor 10

[Chris Plummer]

On Fri, 17 Nov 2023 05:38:28 GMT, Serguei Spitsyn <sspitsyn at 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