RFR: 8286580: serviceability/jvmti/vthread/GetSetLocalTest failed with assert: Not supported for heap frames [v3]

[Serguei Spitsyn]

On Thu, 23 Jun 2022 15:34:11 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> 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 four additional commits since the last revision:
>> 
>>  - resolved 8286580 review comments; problem-listed test GetSetLocalTest under  8286836
>>  - Merge branch 'master' into br2
>>  - remove _depth == 0 clause from sanity check assert
>>  - 8286580: serviceability/jvmti/vthread/GetSetLocalTest failed with assert: Not supported for heap frames
>
> src/hotspot/share/prims/jvmtiImpl.cpp line 654:
> 
>> 652:       // frame at this point, because we aren't truly in the callee yet.
>> 653:       // fr.is_heap_frame() is impossible if a continuation is at a single step or breakpoint.
>> 654:       // In such cases the top frames can't be frozen because of an agent callback frame.
> 
> "frames" should be "frame".  I don't understand the agent callback frame reference.

Removed this sentence as unneeded.

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

PR: https://git.openjdk.org/jdk19/pull/42