On Mon, 12 May 2025 09:35:08 GMT, Per Minborg <pminb...@openjdk.org> wrote:
>> This PR proposes to add a safety net around closing the executor. >> Apparently, in some rare configuration, the VT is not run/notified correctly. >> >> Not completing the test for such a configuration is unlikely to mask >> potential issues that this test is supposed to reveal. > > Per Minborg has updated the pull request incrementally with one additional > commit since the last revision: > > Add shutdown Jai is correct that we can't use System.exit in tests. I added a comment to the JBS issue as it looks like the bug report is with a Zero build. Zero doesn't have VM continuations support so there will be no scheduler and no ForkJoinWorkerThreads. This creates the race where the main thread does the quiescent.notify before the virtual thread waits. Adding `@requires vm.continuations` to the test will skip it for this configuraiton. ------------- PR Comment: https://git.openjdk.org/jdk/pull/25177#issuecomment-2871874731
