On Sun, 9 Aug 2026 00:18:15 GMT, Shiv Shah <[email protected]> wrote:

> Converts suspend001 to ThreadWrapper and fixes the hang. The test assumed 
> MyThread always grabs the shared lock first once main releases it, which 
> isn't guaranteed. With virtual threads the single wakeup goes to the first 
> thread that blocked, that's Suspended, and a suspended virtual thread can't 
> mount, so MyThread was never woken and the debuggee stalled.
> 
> Now each thread gates on its own lock so nothing depends on who gets the 
> wakeup. Also removed the platform only comment and switched thread discovery 
> to the same rendezvous the other converted jdb tests use
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Overall the changes look good, but I think we should hold off on the two 
monitor approach to fixing the deadlock issue. A JVM fix may be coming for 
that, and it would be good to have this test continue to be a test case for the 
fix.

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

PR Review: https://git.openjdk.org/jdk/pull/32267#pullrequestreview-4901253100

Reply via email to