On Wed, 2 Sep 2026 03:12:33 GMT, Shiv Shah <[email protected]> wrote:

>> This change converts the first nsk Suspend/Resume test from the shared agent 
>> thread handshake to direct native calls made by the test thread. The Java 
>> code passes the thread object directly to the native methods, removing the 
>> name-based lookup. Each validation now throws on failure instead of 
>> reporting through the shared status protocol.
>> 
>> The old suspended state check could never actually fail the run, its status 
>> went through a value the test discarded, the rewrite makes all of it load 
>> bearing.
>> 
>> The same schema is still used by sixteen additional tests, plus three tests 
>> that use only its status flag. I plan to convert them family by family, 
>> either in this PR or in follow up PRs depending on what works best for 
>> review. The shared header can be removed once it is no longer used
>> 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Shiv Shah has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Drop the getThreadState wrapper

Thanks for update.

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32585#pullrequestreview-5085238876

Reply via email to