<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Thanks Alan. I’ll port the patch you attached. It’s not a clean port to <b>JDK 17</b>, partly because
<code>UnixDispatcher.java</code> is missing. I’m making the necessary changes manually. Once the build is ready, I’ll validate it and update you on the results.</div>
<div id="appendonsend"></div>
<div style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Alan Bateman <alan.bateman@oracle.com><br>
<b>Sent:</b> Wednesday, March 12, 2025 9:15 PM<br>
<b>To:</b> Shruthi . <Shruthi.Shruthi1@ibm.com>; net-dev@openjdk.org <net-dev@openjdk.org><br>
<b>Cc:</b> Syed Moinudeen <smoinud1@in.ibm.com>; SREENIVAS MAKINEEDI <sreeni@us.ibm.com><br>
<b>Subject:</b> [EXTERNAL] Re: Suggestion needed to port the fix to JDK17 and JDK11S</font>
<div> </div>
</div>
<div>
<div style="display:none!important; display:none; visibility:hidden; font-size:1px; color:#ffffff; line-height:1px; height:0px; max-height:0px; opacity:0; overflow:hidden">
On 07/03/2025 11: 04, Shruthi . wrote: Yes Alan. We have identified the following files that require reordering. src/java. base/share/classes/sun/nio/ch/DatagramChannelImpl. java src/java. base/share/classes/sun/nio/ch/NioSocketImpl. java src/java. base/share/classes/sun/nio/ch/ServerSocketChannelImpl. java</div>
<div style="display:none!important; display:none; visibility:hidden; font-size:1px; color:#ffffff; line-height:1px; max-height:0px; opacity:0; overflow:hidden">
</div>
<style>
<!--
#x_pfptBanneromqt9dg
{display:block!important;
visibility:visible!important;
opacity:1!important;
background-color:#D0D8DC!important;
max-width:none!important;
max-height:none!important}
-->
</style>On 07/03/2025 11:04, Shruthi
. wrote:<br>
<blockquote type="cite"><style type="text/css" style="display:none">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Yes Alan. We have identified the following files that require reordering. <br>
<br>
src/java.base/share/classes/sun/nio/ch/DatagramChannelImpl.java</div>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
src/java.base/share/classes/sun/nio/ch/NioSocketImpl.java</div>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
src/java.base/share/classes/sun/nio/ch/ServerSocketChannelImpl.java</div>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
src/java.base/share/classes/sun/nio/ch/SocketChannelImpl.java</div>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
src/java.base/unix/classes/sun/nio/ch/SinkChannelImpl.java</div>
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
src/java.base/unix/classes/sun/nio/ch/SourceChannelImpl.java<br>
<br>
Please let us know if you think any other files need to be modified</div>
<br>
</blockquote>
The recent refactoring in JDK-8351458 means the change can be more local. Would it be possible to test the patch attached to JDK-8317801 in your environment? This patch is against main line of course.<br>
<br>
-Alan<br>
<br>
[1] <a href="https://bugs.openjdk.org/browse/JDK-8317801" data-auth="NotApplicable" class="x_moz-txt-link-freetext">
https://bugs.openjdk.org/browse/JDK-8317801</a><br>
</div>
</body>
</html>