libceph: kill args in read_partial_message_bio()
authorAlex Elder <elder@inktank.com>
Sat, 9 Mar 2013 00:51:03 +0000 (18:51 -0600)
committerSage Weil <sage@inktank.com>
Thu, 2 May 2013 04:16:32 +0000 (21:16 -0700)
There is only one caller for read_partial_message_bio(), and it
always passes &msg->bio_iter and &bio_seg as the second and third
arguments.  Furthermore, the message in question is always the
connection's in_msg, and we can get that inside the called function.

So drop those two parameters and use their derived equivalents.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
net/ceph/messenger.c

index 962b2cd10f43feab193d6bb2204ba27bc28fa26b..2017b8833baab9f5b02fbe918d5f5e91e693172c 100644 (file)
@@ -1819,14 +1819,16 @@ static int read_partial_message_pages(struct ceph_connection *con,
 
 #ifdef CONFIG_BLOCK
 static int read_partial_message_bio(struct ceph_connection *con,
-                                   struct bio **bio_iter,
-                                   unsigned int *bio_seg,
                                    unsigned int data_len, bool do_datacrc)
 {
-       struct bio_vec *bv = bio_iovec_idx(*bio_iter, *bio_seg);
+       struct ceph_msg *msg = con->in_msg;
+       struct bio_vec *bv;
        void *p;
        int ret, left;
 
+       BUG_ON(!msg);
+       BUG_ON(!msg->bio_iter);
+       bv = bio_iovec_idx(msg->bio_iter, msg->bio_seg);
        left = min((int)(data_len - con->in_msg_pos.data_pos),
                   (int)(bv->bv_len - con->in_msg_pos.page_pos));
 
@@ -1845,7 +1847,7 @@ static int read_partial_message_bio(struct ceph_connection *con,
        con->in_msg_pos.page_pos += ret;
        if (con->in_msg_pos.page_pos == bv->bv_len) {
                con->in_msg_pos.page_pos = 0;
-               iter_bio_next(bio_iter, bio_seg);
+               iter_bio_next(&msg->bio_iter, &msg->bio_seg);
        }
 
        return ret;
@@ -1975,9 +1977,7 @@ static int read_partial_message(struct ceph_connection *con)
                                return ret;
 #ifdef CONFIG_BLOCK
                } else if (m->bio) {
-                       BUG_ON(!m->bio_iter);
                        ret = read_partial_message_bio(con,
-                                                &m->bio_iter, &m->bio_seg,
                                                 data_len, do_datacrc);
                        if (ret <= 0)
                                return ret;