On Fri, 20 Feb 2026 02:29:42 GMT, David Holmes <[email protected]> wrote:

>> Please review the following patch. `ExitOnSuspend::operator()` may run while 
>> a safepoint is in progress, which can lead to crashes because releasing the 
>> monitor involves touching oops if the successor is a virtual thread. The 
>> proposed fix is to process pending operations before releasing the monitor.
>> I added a new test that reliably reproduces the issue with the extra 
>> `assert_not_at_safepoint()` added in this patch, and verified the issue is 
>> now fixed.
>> 
>> Thanks,
>> Patricio
>
> Fix looks good. Thanks
> 
> Nice test! Five Phasers plus a CountDownLatch - wow!

Thanks for the reviews @dholmes-ora and @fbredber!

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

PR Comment: https://git.openjdk.org/jdk/pull/29822#issuecomment-3945663943

Reply via email to