[jdk21] Integrated: 8309612: [REDO] JDK-8307153 JVMTI GetThreadState on carrier should return STATE_WAITING

[Serguei Spitsyn]

On Wed, 21 Jun 2023 02:07:33 GMT, Serguei Spitsyn  wrote:

> Clean backport from latest jdk repo to jdk21 for the fix of:
>   8309612: [REDO] JDK-8307153 JVMTI GetThreadState on carrier should return 
> STATE_WAITING
> 
> Testing:
>  - TBD: mach5 tiers 1-5

This pull request has now been integrated.

Changeset: f8a38eec
Author:Serguei Spitsyn 
URL:   
https://git.openjdk.org/jdk21/commit/f8a38eecc6796ff9671550d45677b06eba8c9372
Stats: 96 lines in 4 files changed: 76 ins; 0 del; 20 mod

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

Reviewed-by: cjplummer
Backport-of: f91e9ba757f04983655c23542e06973805465249

-

PR: https://git.openjdk.org/jdk21/pull/43

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

[Serguei Spitsyn]

On Wed, 7 Jun 2023 18:42:34 GMT, Serguei Spitsyn  wrote:

> 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

This pull request has now been integrated.

Changeset: f91e9ba7
Author:Serguei Spitsyn 
URL:   
https://git.openjdk.org/jdk/commit/f91e9ba757f04983655c23542e06973805465249
Stats: 96 lines in 4 files changed: 76 ins; 0 del; 20 mod

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

Reviewed-by: cjplummer, amenkov

-

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