<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Looks fine Brian. I made a similar change when porting this fix to
    jdk7. Since WinXP was not supported on JDK 8, I didn't see it as
    important. Probably still a good move to fix in jdk8u to keep
    windows XP breathing where possible I guess. Thanks for handling.<br>
    <pre class="moz-signature" cols="72">Regards, 
Sean.</pre>
    <div class="moz-cite-prefix">On 03/09/2015 18:39, Brian Burkhalter
      wrote:<br>
    </div>
    <blockquote
      cite="mid:4CCAA2BB-2E4A-44D8-93E3-D0C4930052C7@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      Please review at your convenience this patch for jdk8u-dev.
      <div><br>
      </div>
      <div>Issue:<span class="Apple-tab-span" style="white-space:pre">  </span><a
          moz-do-not-send="true"
          href="https://bugs.openjdk.java.net/browse/JDK-8133647"><a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8133647">https://bugs.openjdk.java.net/browse/JDK-8133647</a></a></div>
      <div>Patch:<span class="Apple-tab-span" style="white-space:pre">  </span>included
        below</div>
      <div><br>
      </div>
      <div>Summary:</div>
      <div><br>
      </div>
      <div>Ignore WSAEINVAL returned from NET_EnableFastTcpLoopback().
        With this patch the test now succeeds on Windows XP and the
        usual regression test target platforms.</div>
      <div><br>
      </div>
      <div>The change to StressLoopback.java is not very interesting:
        perhaps a “noreg-trivial” label on the issue would be better?</div>
      <div><br>
      </div>
      <div>Subsequent to approval a separate request will be sent to
        jdk8u-dev.</div>
      <div><br>
      </div>
      <div>Thanks,</div>
      <div><br>
      </div>
      <div>Brian</div>
      <div><br>
      </div>
      <div>
        <div style="margin: 0px; font-family: 'Andale Mono';">---
          a/src/windows/native/sun/nio/ch/Net.c</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+++
          b/src/windows/native/sun/nio/ch/Net.c</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">@@ -1,5
          +1,5 @@</div>
        <div style="margin: 0px; font-family: 'Andale Mono';"> /*</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">- *
          Copyright (c) 2001, 2013, Oracle and/or its affiliates. All
          rights reserved.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+ *
          Copyright (c) 2001, 2015, Oracle and/or its affiliates. All
          rights reserved.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * DO NOT
          ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  *</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * This
          code is free software; you can redistribute it and/or modify
          it</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">@@ -157,7
          +157,7 @@</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">        
          if (loopback_available) {</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">          
            int rv = NET_EnableFastTcpLoopback((jint)s);</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">          
            if (rv) {</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">-         
                if (rv == WSAEOPNOTSUPP) {</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+         
                if (rv == WSAEOPNOTSUPP || rv == WSAEINVAL) {</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">          
                    loopback_available = 0;</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">          
                } else {</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">          
                    NET_ThrowNew(env, rv, "fastLoopback");</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">diff --git
          a/test/java/nio/channels/AsynchronousSocketChannel/StressLoopback.java
b/test/java/nio/channels/AsynchronousSocketChannel/StressLoopback.java</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">---
          a/test/java/nio/channels/AsynchronousSocketChannel/StressLoopback.java</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+++
          b/test/java/nio/channels/AsynchronousSocketChannel/StressLoopback.java</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">@@ -1,5
          +1,5 @@</div>
        <div style="margin: 0px; font-family: 'Andale Mono';"> /*</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">- *
          Copyright (c) 2008, 2009, Oracle and/or its affiliates. All
          rights reserved.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+ *
          Copyright (c) 2008, 2015, Oracle and/or its affiliates. All
          rights reserved.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * DO NOT
          ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  *</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * This
          code is free software; you can redistribute it and/or modify
          it</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">@@ -22,7
          +22,7 @@</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  */</div>
        <p style="margin: 0px; font-family: 'Andale Mono'; min-height:
          14px;"> <br class="webkit-block-placeholder">
        </p>
        <div style="margin: 0px; font-family: 'Andale Mono';"> /* @test</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">- * @bug
          6834246 6842687</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">+ * @bug
          6834246 6842687 8133647</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  *
          @summary Stress test connections through the loopback
          interface</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * @run
          main StressLoopback</div>
        <div style="margin: 0px; font-family: 'Andale Mono';">  * @run
          main/othervm -Djdk.net.useFastTcpLoopback StressLoopback</div>
      </div>
    </blockquote>
    <br>
  </body>
</html>