Integrated: 8327650: Test java/nio/channels/DatagramChannel/StressNativeSignal.java timed out
Mark Sheppard
msheppar at openjdk.org
Wed Jun 5 15:51:01 UTC 2024
On Mon, 20 May 2024 14:32:10 GMT, Mark Sheppard <msheppar at openjdk.org> wrote:
> JDK-8327650 Test java/nio/channels/DatagramChannel/StressNativeSignal.java timed out
>
> Please oblige and review the following test only fix to alleviate a race condition within the test.
>
> A race condition exists, in the test StressNativeSignal, due to the setting of a condition variable, shouldTerminate, in the main thread’s invocation of UDPThread::terminate and the setting of shouldTerminate in the main run method of UDPThread. A similar scenario exists for the ServerSocketThread. The variable shouldTerminate is a member variable, and is by default set to false, so the setting to false in the run methods is not required. Removing shouldTerminate = false; statement mitigates the race condition.
>
> Additionally, a number of other minor changes have been added to the test:
> * it is desired to test the asynchronous close of the DatagramChannel::receive method which may trigger the NativeThread::signal issue, thus the while loop has been changed to a do { } while(); loop to invoke the DC receive method.
>
> * The Thread::sleep has been replaced with a CountDownLatch to make the test slightly more deterministic.
>
> It is noted in passing the ServerSocket::run method contains a redundant while loop for reading from an established connection stream, which is never created. This code is never reached as there is no peer to connect to the ServerSocket and the test is on the ServerSocket::accept method.
>
> Another issue of note is that the test is using a hardcoded IANA assigned port 1122 (availant-mgr). But, this has not been altered. The ServerSocket test scenario could use an ephemeral port i.e. new ServerSocket(0);
This pull request has now been integrated.
Changeset: 7acfba28
Author: Mark Sheppard <msheppar at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/7acfba288ff4d1f43cc36506b2bd2d32107b00c2
Stats: 110 lines in 1 file changed: 74 ins; 10 del; 26 mod
8327650: Test java/nio/channels/DatagramChannel/StressNativeSignal.java timed out
Reviewed-by: bpb
-------------
PR: https://git.openjdk.org/jdk/pull/19312
More information about the nio-dev
mailing list