RFR: JDK-8316671: sun/security/ssl/SSLSocketImpl/SSLSocketCloseHang.java test fails intermittent with Read timed out

[Lutz Schmidt]

On Tue, 26 Sep 2023 07:23:20 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> We sometimes run into the following error. This especially occurs when running with fastdebug binaries and on Linux ppc64le machines.
> Current timeout set in the test is 1 second.
> 
> Server accepting: 315357614990101
> Server accepted: 315359219006041
> Client starting handshake: 315359228098300
> java.net.SocketTimeoutException: Read timed out
> at java.base/sun.nio.ch.NioSocketImpl.timedRead(NioSocketImpl.java:278)
> at java.base/sun.nio.ch.NioSocketImpl.implRead(NioSocketImpl.java:304)
> at java.base/sun.nio.ch.NioSocketImpl.read(NioSocketImpl.java:346)
> at java.base/sun.nio.ch.NioSocketImpl$1.read(NioSocketImpl.java:796)
> at java.base/java.net.Socket$SocketInputStream.read(Socket.java:1096)
> at java.base/sun.security.ssl.SSLSocketInputRecord.read(SSLSocketInputRecord.java:489)
> at java.base/sun.security.ssl.SSLSocketInputRecord.readHeader(SSLSocketInputRecord.java:483)
> at java.base/sun.security.ssl.SSLSocketInputRecord.decode(SSLSocketInputRecord.java:160)
> at java.base/sun.security.ssl.SSLTransport.decode(SSLTransport.java:111)
> at java.base/sun.security.ssl.SSLSocketImpl.decode(SSLSocketImpl.java:1507)
> at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(SSLSocketImpl.java:1422)
> at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:455)
> at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:426)
> at SSLSocketCloseHang.doClientSide(SSLSocketCloseHang.java:149)
> at SSLSocketCloseHang.startClient(SSLSocketCloseHang.java:320)
> at SSLSocketCloseHang.<init>(SSLSocketCloseHang.java:246)
> at SSLSocketCloseHang.main(SSLSocketCloseHang.java:232)
> at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
> at java.base/java.lang.reflect.Method.invoke(Method.java:580)
> at com.sun.javatest.regtest.agent.MainWrapper$MainTask.run(MainWrapper.java:138)
> at java.base/java.lang.Thread.run(Thread.java:1570)
> 
> Making the timeout dependent on the timeout factor would give us the chance to influence the test on such slower setups.

Looks good to me. And trivial. And makes a lot of sense.

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

Marked as reviewed by lucy (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15912#pullrequestreview-1644309988