[EXTERNAL] Re: Backporting Keepalive Windows Extended Socket Options (JDK-8308593) to 11u and 17u

[Severin Gehwolf]

Hi,

On Fri, 2023-08-25 at 20:26 +0000, Terry Chow (Simba Technologies Inc)
wrote:
> Hi Severin,
> 
> I forgot to add this in my last reply. From my last email, if that is
> the correct 21u repo and if there aren't any other special details
> for backporting to 21u, could you or someone with JBS account help me
> add an initial "Fix request 21u" to JDK-8308593 to start the backport
> process for 21u?

If you tell us the "Fix Request" comment, then somebody (perhaps
someone from Microsoft with access?) could transplant it to the master
bug and request the 21u backport approval. Please be sure to mention
testing you've done on various platforms and the risk this change
involves if integrated as well as potential follow-up bug fixes that
need to be backported as well.

Thanks,
Severin

> Thanks again,
> Terry
> From: Terry Chow (Simba Technologies Inc) <v-terrychow at microsoft.com>
> Sent: August 24, 2023 11:42 AM
> To: Severin Gehwolf <sgehwolf at redhat.com>;
> jdk-updates-dev at openjdk.org <jdk-updates-dev at openjdk.org>
> Subject: Re: [EXTERNAL] Re: Backporting Keepalive Windows Extended
> Socket Options (JDK-8308593) to 11u and 17u
>  
> Hi Severin,
> 
> Thanks for the feedback. Just to confirm some details, is this the
> correct repo for backports into 21u? I was expecting a "jdk21u-dev"
> repo, similar to 17u and 11u.
> 
> openjdk/jdk21u: https://openjdk.org/projects/jdk-updates (github.com)
> 
> Also, can the guide below be applied to backporting into 21u, the
> process is the same as 17u and 11u? There aren't any specific details
> I need to be aware of to get a backport into 21u right?
> 
> How to contribute or backport a fix - How to contribute or backport a
> fix - OpenJDK Wiki
> 
> Thanks,
> Terry
> 
> From: Severin Gehwolf <sgehwolf at redhat.com>
> Sent: August 24, 2023 1:17 AM
> To: Terry Chow (Simba Technologies Inc) <v-terrychow at microsoft.com>;
> jdk-updates-dev at openjdk.org <jdk-updates-dev at openjdk.org>
> Subject: [EXTERNAL] Re: Backporting Keepalive Windows Extended Socket
> Options (JDK-8308593) to 11u and 17u
>  
> [You don't often get email from sgehwolf at redhat.com. Learn why this
> is important athttps://aka.ms/LearnAboutSenderIdentification ]
> 
> Hi Terry,
> 
> On Thu, 2023-08-24 at 00:54 +0000, Terry Chow (Simba Technologies
> Inc) wrote:
> > Hi maintainers of OpenJDK updates,
> > 
> > I'd like to backport JDK-8308593 (keepalive extended socket options
> > for Windows) to 11u and 17u (maybe 8u later as well). And so, I
> > just want to open discussion on this topic to hear your opinions
> > before I start the backporting process. I did some initial
> > investigation on the backport and I learned the following:
> > 
> > Backporting to 11u requires three additional changes to be made on
> > top of the backport in order for it to work.
> >     1. Lib-jdk.net.gmk will need to be updated to support building
> > extnet for Windows.
> >     2. jdk.net.ExtendedSocketOption will need to add a case to
> > support returning a Windows instance of PlatformSocketOptions.
> >     3. The Windows PlainSocketImpl class needs to have the same
> > code on lines 61 - 96 in the Unix PlainSocketImpl class.
> > Backporting to 17u requires two additional changes (similar to
> > 11u).
> >     1. jdk.net/Lib.gmkwill need to be updated to support building
> > extnet for Windows.
> >     2. jdk.net.ExtendedSocketOption will need to add a case to
> > support returning a Windows instance of PlatformSocketOptions.
> > Let me know if the additional changes are acceptable or if they're
> > disruptive.
> 
> As the change went into JDK 22, I'd first suggest to get this into
> JDK
> 21u which would bring this feature to LTS users. JDK 17u seems OK as
> a
> next step from a brief glance (once it's in 21u). Only then we should
> evaluate JDK 11u and older releases.
> 
> Thanks,
> Severin
>