<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Please review at your convenience. Note that this follows from previous discussions [1, 2].<div><br></div><div>Issue:<span class="Apple-tab-span" style="white-space:pre"> </span><a href="https://bugs.openjdk.java.net/browse/JDK-8071599">https://bugs.openjdk.java.net/browse/JDK-8071599</a></div><div>Patch:<span class="Apple-tab-span" style="white-space:pre"> </span><a href="http://cr.openjdk.java.net/~bpb/8071599/webrev.00/">http://cr.openjdk.java.net/~bpb/8071599/webrev.00/</a><br><div><br></div><div>This patch passes the JPRT job with testset core and onlytests jdk_nio.</div><div><br></div><div>The changes to SocketChannelImpl.{java,c} are intended to guarantee consistent exception behavior across all platforms. If this is not considered important, then those changes could be suppressed and the SocketAdaptor change</div><div><br></div><div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> <span style="color: #bb2ca2">public</span> <span style="color: #bb2ca2">void</span> sendUrgentData(<span style="color: #bb2ca2">int</span> data) <span style="color: #bb2ca2">throws</span> IOException {</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> <span style="color: #bb2ca2">synchronized</span> (sc.blockingLock()) {</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> <span style="color: #bb2ca2">int</span> n = sc.sendOutOfBandData((<span style="color: #bb2ca2">byte</span>) data);</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> <span style="color: #bb2ca2">if</span> (n == <span style="color: #272ad8">0</span>)</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> <span style="color: #bb2ca2">throw</span> <span style="color: #bb2ca2">new</span> IOException(<span style="color: #d12f1b">"Socket buffer full"</span>);</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> }</div><div style="margin: 0px; font-size: 11px; font-family: Menlo;"> }</div></div><div><br></div><div>should suffice.</div><div><br></div><div>Thanks,</div><div><br></div><div>Brian</div><div><br></div><div>[1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031698.html (should have been in nio-dev)</div><div>[2] http://mail.openjdk.java.net/pipermail/nio-dev/2015-February/003042.html</div></div></body></html>