RFR: 8280944: Enable Unix domain sockets in Windows Selector notification mechanism [v3]

[ExE Boss]

On Wed, 2 Feb 2022 12:12:36 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>> Hi,
>> 
>> Could I get the following change reviewed please?
>> 
>> 8280233 temporarily disabled AF_UNIX sockets in the windows pipe implementation due to a Windows bug. We would like to re-enable one internal usage of AF_UNIX pipes in the JDK, for the windows NIO selector notification mechanism since this use case does not involve closing the socket and should therefore not encounter the bug.
>> 
>> I haven't included a regression test as this change will exercise tests that are currently running into TCP resource limitations on windows 10 client systems.
>> 
>> Thanks,
>> Michael
>
> Michael McMahon has updated the pull request incrementally with one additional commit since the last revision:
> 
>   second review comments

src/java.base/windows/classes/sun/nio/ch/PipeImpl.java line 182:

> 180:      */
> 181:     PipeImpl(SelectorProvider sp) throws IOException {
> 182:         this(sp, true, false);

I’m pretty sure that this should actually be:
Suggestion:

        this(sp, /*AF_UNIX*/ false, /*buffering*/ true);

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

PR: https://git.openjdk.java.net/jdk/pull/7302