Integrated: 8337060: Test java/foreign/TestConcurrentClose.java failed: IllegalStateException: SegmentAccessor::doAccess method not being compiled

[Jorn Vernee]

On Thu, 25 Jul 2024 10:58:42 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> We are seeing some failures of this test in our CI, particularly on Windows machines, which are known to starve out certain threads.
> 
> To make sure the compiler threads get a chance to work, I've added `-Xbatch` to the flags with which the test runs (this was missed in the original PR). I've also removed the timeout from the `awaitCompilation` method. The intent is to instead rely on the overall test's timeout, which can also be adjusted with a timeout factor for slower machines, rather than putting an absolute time limit on the compilation.
> 
> Finally, Alan asked me to increase the timeout for the executor shutdown, since there have also been some timeouts there. The current timeout is borrowed from `test/jdk/java/foreign/TestHandshake`, but that test uses a lot less threads, so it can probably get await with waiting just 20 seconds.
> 
> Testing: I ran this test 200 time in Oracle CI, half of the runs were with `-Duse.JTREG_TEST_THREAD_FACTORY=Virtual -XX:-VerifyContinuations` (which is one of the failing configurations).

This pull request has now been integrated.

Changeset: 81ed0287
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/81ed0287175617e6f7ad79dd7036b2fbde27e80a
Stats:     7 lines in 1 file changed: 1 ins; 5 del; 1 mod

8337060: Test java/foreign/TestConcurrentClose.java failed: IllegalStateException: SegmentAccessor::doAccess method not being compiled

Reviewed-by: mcimadamore, alanb

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

PR: https://git.openjdk.org/jdk/pull/20325