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: http://cr.openjdk.java.net/~xuelei/8141651/webrev.00/ In JDK 9, there is a deadlock introduced. One thread can lock SSLSocketImpl instance, and try to lock SSLSocketImpl.writeLock; Another thread may lock SSLSocketImpl.writeLock, and then try to lock SSLSocketImpl instance. A deadlock happens. This fix is trying to remove the latter lock sequence. No new regression test as it is hard to re-produce the issue. Thanks, Xuelei
