On Wed, 17 May 2023 00:22:08 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> This enhancement adds `PopFrame` support for virtual threads. The spec >> defines minimal support that the JVMTI `PopFrame` can be used for a virtual >> thread suspended an an event. >> Actually, the `PopFrame` can supports suspended and mounted virtual threads. >> >> CSR (approved): https://bugs.openjdk.org/browse/JDK-8308001: add PopFrame >> support for virtual threads >> >> Testing: >> New test was developed: `serviceability/vthread/PopFrameTest`. >> Submitted mach5 tiers 1-6 are good. >> TBD: rerun mach5 tiers 1-6 at the end of review again if necessary. > > test/hotspot/jtreg/serviceability/jvmti/vthread/PopFrameTest/PopFrameTest.java > line 241: > >> 239: } >> 240: >> 241: // This method uses PoopFrame on its own thread. It is expected >> to succeed. > > "PoopFrame" -> "PopFrame" Good catch, thanks! Fixed now. :) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14002#discussion_r1196136667
