Integrated: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

[SendaoYan]

On Sun, 26 May 2024 09:27:00 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
>   ObjectMonitorUsage.java failed with `unexpected waiter_count` after [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32.
>   There are two changes in this PR:
> 1. In `JvmtiEnvBase::get_object_monitor_usage` function, change from `java_lang_VirtualThread::is_instance(thread_oop)` to `thread_oop->is_a(vmClasses::BaseVirtualThread_klass())`to support the alternative implementation.
> 2. In `Threads::get_pending_threads(ThreadsList *, int, address)` function of threads.cpp file, change from `java_lang_VirtualThread::is_instance(thread_oop)` to `thread_oop->is_a(vmClasses::BaseVirtualThread_klass())`to support the alternative implementation. This modified function only used by `JvmtiEnvBase::get_object_monitor_usage(JavaThread*, jobject, jvmtiMonitorUsage*)`, so the risk of the modified on threads.cpp file is low.
> 
>   
> 
> Additional testing:
> - [x] linux x86_32 run all testcases in serviceability/jvmti, all testcases run successed expect `serviceability/jvmti/vthread/GetThreadState/GetThreadStateTest.java#default` run failed. This test also run failed before this PR, which has been recorded in [JDK-8333140](https://bugs.openjdk.org/browse/JDK-8333140)
> - [x] linux x86_64 run all testcases in serviceability/jvmti, all testcases run successed.

This pull request has now been integrated.

Changeset: 51b2f806
Author:    SendaoYan <syan at openjdk.org>
Committer: Serguei Spitsyn <sspitsyn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/51b2f80627adc1ca9f8335c3c028109a7018a8be
Stats:     5 lines in 2 files changed: 0 ins; 0 del; 5 mod

8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

Co-authored-by: Jiawei Tang <jwtang at openjdk.org>
Reviewed-by: sspitsyn, alanb, amitkumar

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

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