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

Reply via email to