On Thu, 18 Nov 2021 07:08:15 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> The test fails when the target JavaThread has is_exiting() status. In such a 
>> case the JvmtiExport::cleanup_thread(this) has already made a clean up of 
>> its jvmtiThreadState, so the JavaThread address returned by 
>> _state->get_thread() is 0xbabababababababa.
>> The fix is to add a check for is_exiting() status into handshake closure 
>> do_thread() early.
>> There following handshake closures are fixed by this update:
>>   - UpdateForPopTopFrameClosure
>>  - SetForceEarlyReturn
>>  - SetFramePopClosure
>
> Serguei Spitsyn has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains three additional 
> commits since the last revision:
> 
>  - Merge
>  - remove vmTestbase/nsk/jvmti/PopFrame/popframe011 from ProblemList.txt
>  - fix 8266593: vmTestbase/nsk/jvmti/PopFrame/popframe011 fails with assert

Wouldn't it suffice to just move the assert then?

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

PR: https://git.openjdk.java.net/jdk/pull/6440

Reply via email to