On Wed, Feb 14, 2018 at 7:09 PM, Sowmini Varadhan
<sowmini.varad...@oracle.com> wrote:
> On (02/14/18 18:48), Willem de Bruijn wrote:
>> If the missing break is intentional, no need to respin just for the other
>> minor comments.
> yes the missing break is intentional-  the function returns the
> size of the scatterlist needed for RDMA, and RDS_CMSG_ZCOPY_COOKIE
> (like RDMA_DEST and RDMA_MAP) is meta-data that does not change
> that size.
> I expect to be in the neighborhood of this  code pretty soon, to get
> the additional opimization of passing up the zcopy completion
> as part of recvmsg (see the discussion in
> https://www.mail-archive.com/netdev@vger.kernel.org/msg212788.html)
> I can take care of the other code-cleanup comment suggestions in
> here at that time..

Sounds good.

Reply via email to