RFR: 8280944: Enable Unix domain sockets in Windows Selector notification mechanism [v3]
Michael McMahon
michaelm at openjdk.java.net
Wed Feb 23 12:11:50 UTC 2022
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
Right. The mistake is in the comment. It should say:
"Creates an unbuffered Pipe implementation" rather than
"Creates a Pipe implementation that supports buffering"
I can fix that easily. Changing the default constructor to create a buffered socket would not work with AF_UNIX sockets as they don't have an equivalent to TCP_NODELAY
-------------
PR: https://git.openjdk.java.net/jdk/pull/7302
More information about the nio-dev
mailing list