RFR: 8304147: JVM crash during shutdown when dumping dynamic archive [v2]

[David Holmes]

On Mon, 27 Mar 2023 21:35:19 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Stack: [0x00007f22cfdae000,0x00007f22cfeaf000],  sp=0x00007f22cfead390,  free space=1020k
>> Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)
>> V  [libjvm.so+0xf54cf4]  HandleArea::allocate_handle(oop)+0x214  (handles.cpp:40)
>> V  [libjvm.so+0xab3494]  ClassPrelinker::find_loaded_class(JavaThread*, oop, Symbol*)+0xb4  (handles.inline.hpp:42)
>> V  [libjvm.so+0xab42f2]  ClassPrelinker::maybe_resolve_class(constantPoolHandle, int, JavaThread*)+0xd2  (classPrelinker.cpp:173)
>> V  [libjvm.so+0xab4f0e]  ClassPrelinker::dumptime_resolve_constants(InstanceKlass*, JavaThread*)+0x40e  (classPrelinker.cpp:134)
>> V  [libjvm.so+0x1621479]  MetaspaceShared::link_shared_classes(bool, JavaThread*)+0x2f9  (metaspaceShared.cpp:621)
>> V  [libjvm.so+0xd01658]  DynamicArchive::dump_at_exit(JavaThread*, char const*)+0x318  (dynamicArchive.cpp:392)
>> V  [libjvm.so+0x1064ea6]  before_exit(JavaThread*, bool)+0x86  (java.cpp:445)
>> V  [libjvm.so+0x1ac8468]  Threads::destroy_vm()+0x1b8  (threads.cpp:1085)
>> V  [libjvm.so+0x118be7d]  jni_DestroyJavaVM+0xad  (jni.cpp:3735)
>> 
>> @coleenp I'm going to assume you will say that there is a missing HM in the ClassPrelinker code. If so I will file an issue for that and add it to this PR.
>
> Okay, thank you.  It's good you found the handle.  Yes, you can file another RFE to move the HM down and leave this as is.

Seems there are a couple of paths missing the HM so I will leave as-is and file follow-up issues. Thanks for the review @coleenp !

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13134#discussion_r1149832304