libceph: use local variable cursor instead of &msg->cursor
Use local variable cursor in place of &msg->cursor in read_partial_msg_data() and write_partial_msg_data(). Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
This commit is contained in:
parent
70cf052d0c
commit
343128ce91
1 changed files with 5 additions and 6 deletions
|
@ -1552,8 +1552,8 @@ static int write_partial_message_data(struct ceph_connection *con)
|
||||||
bool need_crc;
|
bool need_crc;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
page = ceph_msg_data_next(&msg->cursor, &page_offset, &length,
|
page = ceph_msg_data_next(cursor, &page_offset, &length,
|
||||||
&last_piece);
|
&last_piece);
|
||||||
ret = ceph_tcp_sendpage(con->sock, page, page_offset,
|
ret = ceph_tcp_sendpage(con->sock, page, page_offset,
|
||||||
length, !last_piece);
|
length, !last_piece);
|
||||||
if (ret <= 0) {
|
if (ret <= 0) {
|
||||||
|
@ -1564,7 +1564,7 @@ static int write_partial_message_data(struct ceph_connection *con)
|
||||||
}
|
}
|
||||||
if (do_datacrc && cursor->need_crc)
|
if (do_datacrc && cursor->need_crc)
|
||||||
crc = ceph_crc32c_page(crc, page, page_offset, length);
|
crc = ceph_crc32c_page(crc, page, page_offset, length);
|
||||||
need_crc = ceph_msg_data_advance(&msg->cursor, (size_t)ret);
|
need_crc = ceph_msg_data_advance(cursor, (size_t)ret);
|
||||||
}
|
}
|
||||||
|
|
||||||
dout("%s %p msg %p done\n", __func__, con, msg);
|
dout("%s %p msg %p done\n", __func__, con, msg);
|
||||||
|
@ -2246,8 +2246,7 @@ static int read_partial_msg_data(struct ceph_connection *con)
|
||||||
if (do_datacrc)
|
if (do_datacrc)
|
||||||
crc = con->in_data_crc;
|
crc = con->in_data_crc;
|
||||||
while (cursor->resid) {
|
while (cursor->resid) {
|
||||||
page = ceph_msg_data_next(&msg->cursor, &page_offset, &length,
|
page = ceph_msg_data_next(cursor, &page_offset, &length, NULL);
|
||||||
NULL);
|
|
||||||
ret = ceph_tcp_recvpage(con->sock, page, page_offset, length);
|
ret = ceph_tcp_recvpage(con->sock, page, page_offset, length);
|
||||||
if (ret <= 0) {
|
if (ret <= 0) {
|
||||||
if (do_datacrc)
|
if (do_datacrc)
|
||||||
|
@ -2258,7 +2257,7 @@ static int read_partial_msg_data(struct ceph_connection *con)
|
||||||
|
|
||||||
if (do_datacrc)
|
if (do_datacrc)
|
||||||
crc = ceph_crc32c_page(crc, page, page_offset, ret);
|
crc = ceph_crc32c_page(crc, page, page_offset, ret);
|
||||||
(void) ceph_msg_data_advance(&msg->cursor, (size_t)ret);
|
(void) ceph_msg_data_advance(cursor, (size_t)ret);
|
||||||
}
|
}
|
||||||
if (do_datacrc)
|
if (do_datacrc)
|
||||||
con->in_data_crc = crc;
|
con->in_data_crc = crc;
|
||||||
|
|
Loading…
Reference in a new issue