adding rsockets support into JDK

Lu, Yingqi at
Fri Nov 30 16:08:13 UTC 2018

Hi Chris,

I have already updated the CSR at

The Javadoc link inside the CSR has already updated with updated content.

I also reattached the updated zip file of the doc.

Please let me know if there is anything still missing.


>-----Original Message-----
>From: Chris Hegarty [mailto:chris.hegarty at]
>Sent: Friday, November 30, 2018 4:36 AM
>To: Alan Bateman <Alan.Bateman at>; Lu, Yingqi
>< at>
>Cc: Viswanathan, Sandhya <sandhya.viswanathan at>; nio-
>dev at; Aundhe, Shirish <shirish.aundhe at>;
>Kaczmarek, Eric <eric.kaczmarek at>
>Subject: Re: adding rsockets support into JDK
>On 30/11/2018 08:43, Alan Bateman wrote:
>> On 30/11/2018 06:07, Lu, Yingqi wrote:
>>> Hi All,
>>> I updated version 21 of the patch at
>>> Javadoc is available at
>>> ry.html
>> You've addressed my comments in the javadoc so they look good.
>> I see you've added a sentence "If the specified protocol family ..."
>> in response to Chris's comment. I don't think we strictly need this in
>> the javadoc as it relates to how subsequent calls to bind or connect
>> behave and those methods already specify how they handle unsupported
>> address types. If you want to keep wording in the factory methods then
>> it will need a few changes to keep is simple. A suggestion for
>penSocketChannel is:
>> "When binding the channel's socket to a local address, or invoking
>> connect to connect channel's socket, the socket address specified to
>> those methods must correspond to the protocol family specified here".
>This wording is good.
>I requested this note since these sockets / channels will be the first such, that
>when created with an INET6 family will disallow IPv4 addressing. While not
>strictly required, I think it is good to call this out clearly.
>For example DatagramChannel does not do this:
>jshell> var dc =
>dc ==> at 7e774085
>jshell> var addr = Inet6Address.getByName("")
>addr ==> /
>jshell> dc.bind(new InetSocketAddress(addr, 0))
>$10 ==> at 7e774085
>P.S. I'm still working through the remainder of the webrev, but I think the
>specification in RdmaSockets and RdmaSocketOptions is good, subject to the
>aforementioned comments. Please update the CSR and I will re-review it.

More information about the nio-dev mailing list