On Tue, 5 May 2026 18:17:03 GMT, Shiv Shah <[email protected]> wrote:

> Converted 14 nsk/jvmti/unit tests from extends Thread to extends 
> ThreadWrapper so they can run with both platform and virtual threads
> 
> For tests with native methods expecting a Thread (GetStackTrace, 
> GetFrameCount, CheckMonitorInfo, RegisterCompletedThread), changed the native 
> signatures to take Thread and pass  .getThread() at the call sites
> 
> earlyretbase is left unconverted its native code does GetObjectClass on the 
> thread to find activeMethod(), so it needs the wrapper class for method 
> lookup but the real Thread for JVMTI operations. That one probably needs a 
> separate approach.
> 
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

This pull request has now been integrated.

Changeset: 125cffd5
Author:    Shiv Shah <[email protected]>
Committer: SendaoYan <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/125cffd599f0d1952241173200c16e644ded13df
Stats:     62 lines in 14 files changed: 14 ins; 0 del; 48 mod

8382273: Update nsk/jvmti/unit tests to use ThreadWrapper

Reviewed-by: cjplummer, sspitsyn

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

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

Reply via email to