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

Alan Bateman alanb at openjdk.java.net
Wed Feb 2 11:46:04 UTC 2022


On Wed, 2 Feb 2022 10:07:47 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:
> 
>   after review comments

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

> 221:             try {
> 222:                 listener = ServerSocketChannel.open(UNIX);
> 223:                 listener.bind(null);

Updating version looks okay, I just wondering if bind can fail, in which case it would leak a socket.

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

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


More information about the nio-dev mailing list