[jdk8u-dev] RFR: 8305165: [8u] ServiceThread::nmethods_do is not called to keep nmethods from being zombied while in the queue [v2]

[Paul Hohensee]

On Fri, 31 Mar 2023 07:01:06 GMT, Joshua Zhu <jzhu at openjdk.org> wrote:

>> Various Crashes in JvmtiExport::post_compiled_method_load were found in
>> our production environment after users started to receive the JVMTI
>> COMPILED_METHOD_LOAD event.
>> ServiceThread::nmethods_do should have been called to keep nmethods from
>> being zombied while in the JvmtiDeferredEventQueue, but
>> JavaThead::nmethods_do is called at present.
>> Make a virtual function call to the correct method to fix this issue.
>> With the fix, various crashes were gone.
>> This issue only exists in 8u.
>
> Joshua Zhu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add 'virtual' in oops_do and nmethods_do declaration in JavaThread

Also noticed you do not have pre-submit testing enabled. Please do so.

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

PR Comment: https://git.openjdk.org/jdk8u-dev/pull/293#issuecomment-1493496875