<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Helvetica;
        panose-1:0 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:Wingdings;
        panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"Andale Mono";
        panose-1:2 11 5 9 0 0 0 0 0 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0cm;
        margin-right:0cm;
        margin-bottom:0cm;
        margin-left:36.0pt;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0cm;
        mso-margin-bottom-alt:auto;
        margin-left:0cm;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.apple-converted-space
        {mso-style-name:apple-converted-space;}
span.EmailStyle19
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
        {page:WordSection1;}
/* List Definitions */
@list l0
        {mso-list-id:2110350300;
        mso-list-type:hybrid;
        mso-list-template-ids:-2108553144 67698689 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l0:level1
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Symbol;}
@list l0:level2
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:"Courier New";}
@list l0:level3
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Wingdings;}
@list l0:level4
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Symbol;}
@list l0:level5
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:"Courier New";}
@list l0:level6
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Wingdings;}
@list l0:level7
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Symbol;}
@list l0:level8
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:"Courier New";}
@list l0:level9
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-18.0pt;
        font-family:Wingdings;}
ol
        {margin-bottom:0cm;}
ul
        {margin-bottom:0cm;}
--></style>
</head>
<body lang="EN-CA" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">Thank you for filing the ticket Brian.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Recently we have seen an increase in the occurrences of this bug. Roughly 2-3 times per week for some of our applications/roles/servers. I have attempted to make the test program a little simpler and removed the reflection hacks. I have
 attached the new version here. I am able to reproduce this issue on the following platforms:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<ul style="margin-top:0cm" type="disc">
<li class="MsoListParagraph" style="margin-left:0cm;mso-list:l0 level1 lfo1">Linux - java version "1.8.0_172" - build 1.8.0_172-b11<o:p></o:p></li><li class="MsoListParagraph" style="margin-left:0cm;mso-list:l0 level1 lfo1">Linux - java version "9.0.4" - build 9.0.4+11<o:p></o:p></li><li class="MsoListParagraph" style="margin-left:0cm;mso-list:l0 level1 lfo1">Linux - java version "10" 2018-03-20 - build 10+46<o:p></o:p></li><li class="MsoListParagraph" style="margin-left:0cm;mso-list:l0 level1 lfo1">Mac - java version "1.8.0_131" - build 1.8.0_131-b11<o:p></o:p></li></ul>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Additionally I have taken a closer look at EPollPort.java and feel like the following change may address is issue:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">--- a/src/java.base/linux/classes/sun/nio/ch/EPollPort.java<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+++ b/src/java.base/linux/classes/sun/nio/ch/EPollPort.java<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">@@ -85,6 +85,7 @@ final class EPollPort<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">     private final ArrayBlockingQueue<Event> queue;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">     private final Event NEED_TO_POLL = new Event(null, 0);<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">     private final Event EXECUTE_TASK_OR_SHUTDOWN = new Event(null, 0);<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+    private final Event NOOP = new Event(null, 0);<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono""><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">     EPollPort(AsynchronousChannelProvider provider, ThreadPool pool)<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">         throws IOException<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">@@ -194,7 +195,6 @@ final class EPollPort<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">     private class EventHandlerTask implements Runnable {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">         private Event poll() throws IOException {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">             try {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">-                for (;;) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     int n;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     do {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                         n = EPoll.wait(epfd, address, MAX_EPOLL_EVENTS, -1);<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">@@ -250,7 +250,8 @@ final class EPollPort<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     } finally {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                         fdToChannelLock.readLock().unlock();<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     }<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">-                }<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                    // There is no real event to return. So return NOOP.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                    return NOOP;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">             } finally {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                 // to ensure that some thread will poll when all events have<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                 // been consumed<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">@@ -288,6 +289,11 @@ final class EPollPort<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                         continue;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     }<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono""><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                    // there is nothing to do<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                    if (ev == NOOP) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                        continue;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+                    }<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">+<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     // handle wakeup to execute task or shutdown<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                     if (ev == EXECUTE_TASK_OR_SHUTDOWN) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Andale Mono"">                         Runnable task = pollTask();</span><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Removal of the for(;;) loop I believe is the key important change because with that loop it is possible to overflow the queue, and to lose IO events. What do you think?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">Miah<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">Brian Burkhalter <brian.burkhalter@oracle.com><br>
<b>Date: </b>Thursday, January 12, 2017 at 8:06 PM<br>
<b>To: </b>Jeremiah Ness <jness@proofpoint.com><br>
<b>Cc: </b>Alan Bateman <Alan.Bateman@oracle.com>, "nio-dev@openjdk.java.net" <nio-dev@openjdk.java.net><br>
<b>Subject: </b>Re: all EventHandlerTasks in EPollPort waiting on queue<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">On Jan 12, 2017, at 8:03 AM, Jeremiah Ness <<a href="mailto:jness@proofpoint.com">jness@proofpoint.com</a>> wrote:<o:p></o:p></p>
<div>
<p class="MsoNormal"><br>
<br>
<o:p></o:p></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt;orphans: auto;text-align:start;widows: auto;-webkit-text-stroke-width: 0px;word-spacing:0px">
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<p class="MsoNormal"><span style="font-size:9.0pt;font-family:Helvetica">There does appear to be an issue here. Can you create a standalone test<span class="apple-converted-space"> </span><br>
case to tickle test so that we can include it in a bug report? That<span class="apple-converted-space"> </span><br>
would really help get to a regression test to include with the fix.<o:p></o:p></span></p>
</blockquote>
<p class="MsoNormal"><span style="font-size:9.0pt;font-family:Helvetica"><br>
I do have a test program that can trigger the condition for me. I have attached the source code for PortTest.java. PortTest has successfully demonstrated the issue in the following configurations:<br>
<br>
- OSX 10.11.6 with java version "1.8.0_112"<br>
- CentOs7 with kernel 3.10.0-514.2.2.el7.x86_64 and with openjdk version "1.8.0_111"</span><o:p></o:p></p>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">I have filed <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8172750&d=DwMFAg&c=Vxt5e0Osvvt2gflwSlsJ5DmPGcPvTRKLJyp031rXjhg&r=P6hDcvKJR-XgpCXh6ZenEnMYTyDmTBSvESHlaoeZnug&m=OBtwHBe_CbECkxajzDac8d2fEXQoiSRM8ZOOYYGBCts&s=6tmxao5LJNtHp_ow8xE9XWclmF6481abYh4VBflMxso&e=">https://bugs.openjdk.java.net/browse/JDK-8172750</a>
 to track this problem. Thank you for providing a test case.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Brian<o:p></o:p></p>
</div>
</div>
</body>
</html>