Re: [PATCH net-next v4 03/15] ceph: Use sendmsg(MSG_SPLICE_PAGES) rather than sendpage

From: David Howells
Date: Fri Jun 23 2023 - 15:44:46 EST


Jakub Kicinski <kuba@xxxxxxxxxx> wrote:

> > if (length == cursor->total_resid)
> > - more = MSG_MORE;
> > - ret = ceph_tcp_sendpage(con->sock, page, page_offset, length,
> > - more);
> > + msghdr.msg_flags |= MSG_MORE;
>
> Should the condition also be flipped here, like you did below?
> (can be a follow up if so)

Yeah - the "==" in the if-statement needs flipping to "!=". I can send a
follow-up patch for it or respin if you prefer.

David