<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<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);">
Hey Severin,</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
My mistake for not including the fix request comment in the last email. Also, last week I tried reaching out to my Microsoft JDK contact for help with this, but it seems like they're away at the moment without a return date.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Here's the "Fix request" comment, in case you or anyone else can help apply the request to ticket JDK-8308593:</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<i>Fix Request 21u</i></div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<i><br>
</i></div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<i>The reason for backporting the Windows keepalive extended socket options to 21u is to have keepalive parity with the other OS platforms. The risk is low as the backported code introduces a new code path that only takes effect when keepalive extended socket
options are set on a Windows platform. It isn't expected to cause unintended effects or affect any current users as the keepalive options were never supported in the JDK for Windows before this patch. Tier 1 tests were ran and they're passing. I also manually
tested and confirmed the patch to be working on Windows (specifically Windows 10 since that's the only Windows platform available to me at the moment). Test coverage is covered by existing tests from another backport from JDK-8194298. The backport cleanly
applies to 21u without any conflicts.</i></div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Thanks again Severin,</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Terry</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Severin Gehwolf <sgehwolf@redhat.com><br>
<b>Sent:</b> August 28, 2023 5:50 AM<br>
<b>To:</b> Terry Chow (Simba Technologies Inc) <v-terrychow@microsoft.com>; jdk-updates-dev@openjdk.org <jdk-updates-dev@openjdk.org><br>
<b>Subject:</b> Re: [EXTERNAL] Re: Backporting Keepalive Windows Extended Socket Options (JDK-8308593) to 11u and 17u</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hi,<br>
<br>
On Fri, 2023-08-25 at 20:26 +0000, Terry Chow (Simba Technologies Inc)<br>
wrote:<br>
> Hi Severin,<br>
> <br>
> I forgot to add this in my last reply. From my last email, if that is<br>
> the correct 21u repo and if there aren't any other special details<br>
> for backporting to 21u, could you or someone with JBS account help me<br>
> add an initial "Fix request 21u" to JDK-8308593 to start the backport<br>
> process for 21u?<br>
<br>
If you tell us the "Fix Request" comment, then somebody (perhaps<br>
someone from Microsoft with access?) could transplant it to the master<br>
bug and request the 21u backport approval. Please be sure to mention<br>
testing you've done on various platforms and the risk this change<br>
involves if integrated as well as potential follow-up bug fixes that<br>
need to be backported as well.<br>
<br>
Thanks,<br>
Severin<br>
<br>
> Thanks again,<br>
> Terry<br>
> From: Terry Chow (Simba Technologies Inc) <v-terrychow@microsoft.com><br>
> Sent: August 24, 2023 11:42 AM<br>
> To: Severin Gehwolf <sgehwolf@redhat.com>;<br>
> jdk-updates-dev@openjdk.org <jdk-updates-dev@openjdk.org><br>
> Subject: Re: [EXTERNAL] Re: Backporting Keepalive Windows Extended<br>
> Socket Options (JDK-8308593) to 11u and 17u<br>
> <br>
> Hi Severin,<br>
> <br>
> Thanks for the feedback. Just to confirm some details, is this the<br>
> correct repo for backports into 21u? I was expecting a "jdk21u-dev"<br>
> repo, similar to 17u and 11u.<br>
> <br>
> openjdk/jdk21u: <a href="https://openjdk.org/projects/jdk-updates">https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fopenjdk.org%2Fprojects%2Fjdk-updates&data=05%7C01%7Cv-terrychow%40microsoft.com%7Ca355ccafb8574a17b61108dba7c561b6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C638288238420042142%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=MmdGW1%2Fnx%2F6Qmb06WZBInKfmwjr8%2BEWNPk6%2FNmk4FtU%3D&reserved=0</a>
(github.com)<br>
> <br>
> Also, can the guide below be applied to backporting into 21u, the<br>
> process is the same as 17u and 11u? There aren't any specific details<br>
> I need to be aware of to get a backport into 21u right?<br>
> <br>
> How to contribute or backport a fix - How to contribute or backport a<br>
> fix - OpenJDK Wiki<br>
> <br>
> Thanks,<br>
> Terry<br>
> <br>
> From: Severin Gehwolf <sgehwolf@redhat.com><br>
> Sent: August 24, 2023 1:17 AM<br>
> To: Terry Chow (Simba Technologies Inc) <v-terrychow@microsoft.com>;<br>
> jdk-updates-dev@openjdk.org <jdk-updates-dev@openjdk.org><br>
> Subject: [EXTERNAL] Re: Backporting Keepalive Windows Extended Socket<br>
> Options (JDK-8308593) to 11u and 17u<br>
> <br>
> [You don't often get email from sgehwolf@redhat.com. Learn why this<br>
> is important athttps://aka.ms/LearnAboutSenderIdentification ]<br>
> <br>
> Hi Terry,<br>
> <br>
> On Thu, 2023-08-24 at 00:54 +0000, Terry Chow (Simba Technologies<br>
> Inc) wrote:<br>
> > Hi maintainers of OpenJDK updates,<br>
> > <br>
> > I'd like to backport JDK-8308593 (keepalive extended socket options<br>
> > for Windows) to 11u and 17u (maybe 8u later as well). And so, I<br>
> > just want to open discussion on this topic to hear your opinions<br>
> > before I start the backporting process. I did some initial<br>
> > investigation on the backport and I learned the following:<br>
> > <br>
> > Backporting to 11u requires three additional changes to be made on<br>
> > top of the backport in order for it to work.<br>
> > 1. Lib-jdk.net.gmk will need to be updated to support building<br>
> > extnet for Windows.<br>
> > 2. jdk.net.ExtendedSocketOption will need to add a case to<br>
> > support returning a Windows instance of PlatformSocketOptions.<br>
> > 3. The Windows PlainSocketImpl class needs to have the same<br>
> > code on lines 61 - 96 in the Unix PlainSocketImpl class.<br>
> > Backporting to 17u requires two additional changes (similar to<br>
> > 11u).<br>
> > 1. jdk.net/Lib.gmkwill need to be updated to support building<br>
> > extnet for Windows.<br>
> > 2. jdk.net.ExtendedSocketOption will need to add a case to<br>
> > support returning a Windows instance of PlatformSocketOptions.<br>
> > Let me know if the additional changes are acceptable or if they're<br>
> > disruptive.<br>
> <br>
> As the change went into JDK 22, I'd first suggest to get this into<br>
> JDK<br>
> 21u which would bring this feature to LTS users. JDK 17u seems OK as<br>
> a<br>
> next step from a brief glance (once it's in 21u). Only then we should<br>
> evaluate JDK 11u and older releases.<br>
> <br>
> Thanks,<br>
> Severin<br>
> <br>
<br>
</div>
</span></font></div>
</body>
</html>