RFR: 8308593: Add Keepalive Extended Socket Options Support for Windows [v2]

Terry Chow duke at openjdk.org
Fri Jun 16 17:19:28 UTC 2023


> The PR adds support for the keepalive extended socket options on Windows. For TCP_KEEPIDLE and TCP_KEEPINTVL, these options are supported starting from Windows 10 version 1709. TCP_KEEPCNT is supported starting from Windows 10 version 1703. Information on these socket options can be found [here](https://learn.microsoft.com/en-us/windows/win32/winsock/ipproto-tcp-socket-options).
> 
> I've also corrected the `handleError()` function. On Windows, the error needs to be retrieved using `WSAGetLastError()` and error codes are prefixed with "WSA". Information on this can be found [here](https://learn.microsoft.com/en-us/windows/win32/winsock/error-codes-errno-h-errno-and-wsagetlasterror-2).
> 
>>The error codes returned by Windows Sockets are similar to UNIX socket error code constants, but the constants are all prefixed with WSA.
> 
>>Error codes set by Windows Sockets are not made available through the errno variable.
> 
> No new tests were added as the existing tests should cover this.

Terry Chow has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision:

  Support keepalive extended socket options for Windows

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/14232/files
  - new: https://git.openjdk.org/jdk/pull/14232/files/a7858982..2ada2207

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14232&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14232&range=00-01

  Stats: 78650 lines in 1390 files changed: 57268 ins; 16375 del; 5007 mod
  Patch: https://git.openjdk.org/jdk/pull/14232.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14232/head:pull/14232

PR: https://git.openjdk.org/jdk/pull/14232


More information about the net-dev mailing list