RFR: 8269530: runtime/ParallelLoad/ParallelSuperTest.java timeout

Coleen Phillimore coleenp at openjdk.java.net
Tue Jun 29 13:04:05 UTC 2021


On Mon, 28 Jun 2021 16:18:00 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Hi,
> 
> Please review this small fix to test ParallelSuperTest.java. I only saw the timeout once on one of my GHA tests but the issue can be easily reproduced by adding a delay in startLoading() before the main thread tries to synchronize on thread_sync.
> Run the test with and without the added delay. The test times out without the patch and passes with the patch.
> 
> Thanks,
> Patricio

Thank you so much for fixing this! I guessed that the synchronization was dubious when I wrote the test but it seemed to do what I want wrt timing, so I left it.  That's how it got checked in.  Thanks for fixing it.

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

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4614


More information about the hotspot-runtime-dev mailing list