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