RFR: 8309612: [REDO] JDK-8307153 JVMTI GetThreadState on carrier should return STATE_WAITING [v3]

Serguei Spitsyn sspitsyn at openjdk.org
Wed Jun 7 19:32:23 UTC 2023


> This is REDO the fix of [JDK-8307153](https://bugs.openjdk.org/browse/JDK-8307153).
> The last update of the fix in the review cycle was incorrect and incorrectly tested, so the issue has not been noticed. It is why the fix was backed out.
> The issue is that the SUSPEND bit was missed in the JVMTI thread state of platform/carrier threads carrying virtual threads (see`JvmtiEnvBase::get_thread_state` function).
> 
> The first push/patch is the original fix of JDK-8307153.
> The fix of the SUSPEND bit issue will be in the incremental update.
> It is to simplify the review.
> 
> Testing:
>  - TBD: mach5 tiers 1-5

Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:

  cleanup in comments: replace confusing term: passive carrier thread

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/14366/files
  - new: https://git.openjdk.org/jdk/pull/14366/files/29adb0af..094b5f28

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14366&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14366&range=01-02

  Stats: 3 lines in 2 files changed: 0 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/14366.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14366/head:pull/14366

PR: https://git.openjdk.org/jdk/pull/14366


More information about the serviceability-dev mailing list