adding rsockets support into JDK
Brian Burkhalter
brian.burkhalter at oracle.com
Mon Feb 4 21:58:03 UTC 2019
Hi Lucy,
I found and fixed some small errors in the RdmaSockets doc:
http://hg.openjdk.java.net/jdk/sandbox/rev/78d016915473
I also have some doubts about the accuracy of the documentation in this class regarding socket options. Firstly, the options supported by the objects returned by openSocket() and openSocketChannel() look as if they do not include *all* the options supported by java.net.Socket and java.net.ServerSocket, respectively, contrary to the documentation, e.g.,
"An RDMA socket supports the same socket options that java.net.Socket <file:///Users/bpb/Work/CoreLibs/jdk/sandbox/build/macosx-x86_64-server-release/images/docs/api/java.base/java/net/Socket.html> defines. In addition, it supports the socket options specified by RdmaSocketOptions <file:///Users/bpb/Work/CoreLibs/jdk/sandbox/build/macosx-x86_64-server-release/images/docs/api/jdk.net/jdk/net/RdmaSocketOptions.html>.”.
In addition to the RDMA socket options, the RDMA socket and socket channel appear to support SO_SNDBUF, SO_RCVBUF, SO_REUSEADDR, and TCP_NODELAY which are only a proper subset of the options supported by Socket{Channel} and what one would expect based on [1].
Secondly, openServerSocketChannel() does not include the sentence
"In addition, it supports the socket options specified by RdmaSocketOptions <file:///Users/bpb/Work/CoreLibs/jdk/sandbox/build/macosx-x86_64-server-release/images/docs/api/jdk.net/jdk/net/RdmaSocketOptions.html>.”
but looks like it should.
Thirdly, the documentation of openServerSocketChannel() does not mention options at all which appears to be an omission.
I am happy to update this documentation as needed once there is corroboration or rejection of my foregoing comments.
Thanks,
Brian
[1] https://linux.die.net/man/7/rsocket
> On Feb 4, 2019, at 11:51 AM, Lu, Yingqi <yingqi.lu at intel.com> wrote:
>
> I think we already addressed all the feedbacks. However, Alan/Chris, please let us know if there is anything else we need to change in terms of JavaDoc wording.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20190204/7341997e/attachment-0001.html>
More information about the nio-dev
mailing list