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

[Terry Chow (Simba Technologies Inc)]

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)<https://github.com/openjdk/jdk21u>

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<https://wiki.openjdk.org/display/JDKUpdates/How+to+contribute+or+backport+a+fix>

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 at https://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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/jdk-updates-dev/attachments/20230824/c1acf9fd/attachment.htm>