Re: Code review request, 8013809 deadlock in SSLSocketImpl between between write and close

2013-08-05 Thread Xuelei Fan
If no objections, I will push the changeset, and request to back the fix. Thanks, Xuelei On 8/2/2013 9:12 AM, Xuelei Fan wrote: > On 8/2/2013 8:45 AM, Brad Wetmore wrote: >> >> >> On 7/25/2013 2:11 AM, Xuelei Fan wrote: >>> Hi Brad, >>> >>> Are you available to review this fix? >>> >>> Webrev: ht

Re: Code review request, 8013809 deadlock in SSLSocketImpl between between write and close

2013-08-01 Thread Xuelei Fan
On 8/2/2013 8:45 AM, Brad Wetmore wrote: > > > On 7/25/2013 2:11 AM, Xuelei Fan wrote: >> Hi Brad, >> >> Are you available to review this fix? >> >> Webrev: http://cr.openjdk.java.net/~xuelei/8013809/webrev.00/ >> >> No new regression test, hard to reproduce the issue. > > Your immediate fix loo

Re: Code review request, 8013809 deadlock in SSLSocketImpl between between write and close

2013-08-01 Thread Brad Wetmore
On 7/25/2013 2:11 AM, Xuelei Fan wrote: Hi Brad, Are you available to review this fix? Webrev: http://cr.openjdk.java.net/~xuelei/8013809/webrev.00/ No new regression test, hard to reproduce the issue. Your immediate fix looks good, however, IIRC, the reason for having getConnectionState(

Code review request, 8013809 deadlock in SSLSocketImpl between between write and close

2013-07-25 Thread Xuelei Fan
Hi Brad, Are you available to review this fix? Webrev: http://cr.openjdk.java.net/~xuelei/8013809/webrev.00/ No new regression test, hard to reproduce the issue. Thanks, Xuelei