diff options
author | Benoît Canet <benoit.canet@nodalink.com> | 2015-06-25 20:32:34 +0300 |
---|---|---|
committer | Ilya Dryomov <idryomov@gmail.com> | 2015-06-29 20:03:46 +0300 |
commit | c2cfa19400979dc1a14bba75f83b451b0cd9507a (patch) | |
tree | be54f525471187bb09e130bdd9290e87015ba785 /kernel/locking | |
parent | 6ba8edc0bcbdf337293e60123ddac8fc1c895a3c (diff) | |
download | op-kernel-dev-c2cfa19400979dc1a14bba75f83b451b0cd9507a.zip op-kernel-dev-c2cfa19400979dc1a14bba75f83b451b0cd9507a.tar.gz |
libceph: Fix ceph_tcp_sendpage()'s more boolean usage
From struct ceph_msg_data_cursor in include/linux/ceph/messenger.h:
bool last_piece; /* current is last piece */
In ceph_msg_data_next():
*last_piece = cursor->last_piece;
A call to ceph_msg_data_next() is followed by:
ret = ceph_tcp_sendpage(con->sock, page, page_offset,
length, last_piece);
while ceph_tcp_sendpage() is:
static int ceph_tcp_sendpage(struct socket *sock, struct page *page,
int offset, size_t size, bool more)
The logic is inverted: correct it.
Signed-off-by: Benoît Canet <benoit.canet@nodalink.com>
Reviewed-by: Alex Elder <elder@linaro.org>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'kernel/locking')
0 files changed, 0 insertions, 0 deletions