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).

Looks good.

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

Marked as reviewed by cjplummer (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/31043#pullrequestreview-4285204831

Reply via email to