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