RFR: 8254668: JVMTI process frames on thread without started processing [v2]

Stefan Karlsson stefank at openjdk.java.net
Tue Oct 13 13:06:30 UTC 2020


On Tue, 13 Oct 2020 12:48:05 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

>> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review 1
>
> Hi Stefan,
> 
> thanks for fixing.
> 
> With this change the assertion in pr #119 does not fail anymore.
> 
> The fix looks good to me but I'm not an ZGC expert, neither a Reviewer :)

Thanks @fisk and @reinrich for reviewing.

> src/hotspot/share/prims/jvmtiEnvBase.cpp line 559:
> 
>> 557:
>> 558: // return the vframe on the specified thread and depth, NULL if no such frame
>> 559: // The thread and the oops in the returned might not have been process.
> 
> s/the returned/the returned vframe/

Thanks for noticing.

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

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


More information about the serviceability-dev mailing list