Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v3]

2021-07-08 Thread Xue-Lei Andrew Fan
On Wed, 7 Jul 2021 20:44:25 GMT, Alexey Bakhtin wrote: >> Please review the fix for JDK-8268965. >> >> The new jtreg test is added for the described issue. >> sun/security/ssl and javax/net/ssl tests are passed > > Alexey Bakhtin has updated the pull request incrementally with one additional >

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v2]

2021-07-07 Thread Alexey Bakhtin
On Wed, 7 Jul 2021 17:22:07 GMT, Xue-Lei Andrew Fan wrote: >> Alexey Bakhtin has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add read lock for inputRecord.deplete > > src/java.base/share/classes/sun/security/ssl/SSLSocketImpl.java line 1

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v3]

2021-07-07 Thread Alexey Bakhtin
> Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed Alexey Bakhtin has updated the pull request incrementally with one additional commit since the last revision: Use tryLock - Cha

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v2]

2021-07-07 Thread Xue-Lei Andrew Fan
On Wed, 7 Jul 2021 10:15:19 GMT, Alexey Bakhtin wrote: >> Please review the fix for JDK-8268965. >> >> The new jtreg test is added for the described issue. >> sun/security/ssl and javax/net/ssl tests are passed > > Alexey Bakhtin has updated the pull request incrementally with one additional >

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v2]

2021-07-07 Thread Alexey Bakhtin
On Wed, 7 Jul 2021 10:15:19 GMT, Alexey Bakhtin wrote: >> Please review the fix for JDK-8268965. >> >> The new jtreg test is added for the described issue. >> sun/security/ssl and javax/net/ssl tests are passed > > Alexey Bakhtin has updated the pull request incrementally with one additional >

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server [v2]

2021-07-07 Thread Alexey Bakhtin
> Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed Alexey Bakhtin has updated the pull request incrementally with one additional commit since the last revision: Add read lock for inputRecord.d

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server

2021-07-06 Thread Xue-Lei Andrew Fan
On Thu, 17 Jun 2021 13:20:54 GMT, Alexey Bakhtin wrote: > Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed Hi Alexey, Thank you for the details thread stacks. The basic idea of the fix looks g

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server

2021-07-06 Thread Alexey Bakhtin
On Thu, 17 Jun 2021 13:20:54 GMT, Alexey Bakhtin wrote: > Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed Hi Xuelei, Thank you for your interest in this issue. In this particular test case, a s

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server

2021-07-05 Thread Xue-Lei Andrew Fan
On Thu, 17 Jun 2021 13:20:54 GMT, Alexey Bakhtin wrote: > Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed Could I have more details about the problem? For example, the thread stacks for the s

Re: RFR: 8268965: TCP Connection Reset when connecting simple socket to SSL server

2021-06-29 Thread Xue-Lei Andrew Fan
On Thu, 17 Jun 2021 13:20:54 GMT, Alexey Bakhtin wrote: > Please review the fix for JDK-8268965. > > The new jtreg test is added for the described issue. > sun/security/ssl and javax/net/ssl tests are passed I would like to have a look. I was wondering if there is a racing between close and r