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