On Thu, 10 Jul 2025 10:02:56 GMT, Richard Reingruber <rr...@openjdk.org> wrote:

>> This pr moves the `notify()` call to the finally block to avoid the deadlock 
>> in `join()` if `verifyHeapDump(dumpFile)` throws an exception.
>> 
>> Testing was done with fastdebug and release builds on the main platforms and 
>> also on Linux/PPC64le and AIX.
>
> Richard Reingruber has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Wait while !done

With your fix in place, I assume now if verifyHeapDump() throws an exception 
the test will still fail, but in a more timely and meaningful manner. Why is 
verifyHeapDump() throwing an exception in the first place. Is that something 
that still needs to be addressed?

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

PR Comment: https://git.openjdk.org/jdk/pull/26213#issuecomment-3059069045

Reply via email to