On Tue, 14 Mar 2023 12:23:08 GMT, Markus Grönlund <mgron...@openjdk.org> wrote:
>> src/hotspot/share/prims/agentList.cpp line 419: >> >>> 417: const jint err = (*on_load_entry)(&main_vm, >>> const_cast<char*>(agent->options()), NULL); >>> 418: if (err != JNI_OK) { >>> 419: vm_exit_during_initialization("-Xrun library failed to init", >>> agent->name()); >> >> Do you need to be back in `_thread_in_vm` before exiting? > > Hmm. This was ported as is. I will double-check. Looks like there is no requirement to be in _thread_in_vm before invoking vm_exit_during_initialization(). vm_perform_shutdown_actions() will forcibly set the thread state to _thread_in_native (no transition). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/12923#discussion_r1153243069