summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorColy Li <colyli@suse.de>2020-10-02 16:27:34 +0800
committerDavid S. Miller <davem@davemloft.net>2020-10-02 15:27:08 -0700
commit40efc4dc73956e1fab177783e55f30117517c542 (patch)
treeb9afe6c323f391becaa8b69706659bfb58703bc2 /net
parent6aa25c737705466020e74ad44679d3758b8961f6 (diff)
downloadlinux-40efc4dc73956e1fab177783e55f30117517c542.tar.bz2
libceph: use sendpage_ok() in ceph_tcp_sendpage()
In libceph, ceph_tcp_sendpage() does the following checks before handle the page by network layer's zero copy sendpage method, if (page_count(page) >= 1 && !PageSlab(page)) This check is exactly what sendpage_ok() does. This patch replace the open coded checks by sendpage_ok() as a code cleanup. Signed-off-by: Coly Li <colyli@suse.de> Acked-by: Jeff Layton <jlayton@kernel.org> Cc: Ilya Dryomov <idryomov@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/ceph/messenger.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c
index bdfd66ba3843..d4d7a0e52491 100644
--- a/net/ceph/messenger.c
+++ b/net/ceph/messenger.c
@@ -575,7 +575,7 @@ static int ceph_tcp_sendpage(struct socket *sock, struct page *page,
* coalescing neighboring slab objects into a single frag which
* triggers one of hardened usercopy checks.
*/
- if (page_count(page) >= 1 && !PageSlab(page))
+ if (sendpage_ok(page))
sendpage = sock->ops->sendpage;
else
sendpage = sock_no_sendpage;