Re: Code Review Request, 8141651 Deadlock in sun.security.ssl.SSLSocketImpl

2015-12-10 Thread Bradford Wetmore
Looks good. Brad On 12/9/2015 11:40 AM, Sean Mullan wrote: Fix looks good, interesting issue, though I wonder if there is a better locking scheme, but probably a question for another time. --Sean On 12/05/2015 07:03 PM, Xuelei Fan wrote: Hi, Please review the fix for JDK-8141651: ht

Re: Code Review Request, 8141651 Deadlock in sun.security.ssl.SSLSocketImpl

2015-12-09 Thread Sean Mullan
Fix looks good, interesting issue, though I wonder if there is a better locking scheme, but probably a question for another time. --Sean On 12/05/2015 07:03 PM, Xuelei Fan wrote: Hi, Please review the fix for JDK-8141651: http://cr.openjdk.java.net/~xuelei/8141651/webrev.00/ In JDK 9,