libceph: drop pages parameter
authorAlex Elder <elder@inktank.com>
Tue, 12 Mar 2013 04:34:22 +0000 (23:34 -0500)
committerSage Weil <sage@inktank.com>
Thu, 2 May 2013 04:17:23 +0000 (21:17 -0700)
The value passed for "pages" in read_partial_message_pages() is
always the pages pointer from the incoming message, which can be
derived inside that function.  So just get rid of the parameter.

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

index 0a9f6362d4d8bfa245f480193157968b4522429d..95f90b01f7531f84064c0a2e919430c5afd1c914 100644 (file)
@@ -2203,10 +2203,11 @@ static int read_partial_message_section(struct ceph_connection *con,
 static int ceph_con_in_msg_alloc(struct ceph_connection *con, int *skip);
 
 static int read_partial_message_pages(struct ceph_connection *con,
-                                     struct page **pages,
                                      unsigned int data_len, bool do_datacrc)
 {
+       struct ceph_msg *msg = con->in_msg;
        struct ceph_msg_pos *msg_pos = &con->in_msg_pos;
+       struct page **pages;
        struct page *page;
        size_t page_offset;
        size_t length;
@@ -2214,6 +2215,7 @@ static int read_partial_message_pages(struct ceph_connection *con,
        int ret;
 
        /* (page) data */
+       pages = msg->p.pages;
        BUG_ON(pages == NULL);
        page = pages[msg_pos->page];
        page_offset = msg_pos->page_pos;
@@ -2285,8 +2287,8 @@ static int read_partial_msg_data(struct ceph_connection *con)
        data_len = le32_to_cpu(con->in_hdr.data_len);
        while (msg_pos->data_pos < data_len) {
                if (ceph_msg_has_pages(msg)) {
-                       ret = read_partial_message_pages(con, msg->p.pages,
-                                                data_len, do_datacrc);
+                       ret = read_partial_message_pages(con, data_len,
+                                                       do_datacrc);
                        if (ret <= 0)
                                return ret;
 #ifdef CONFIG_BLOCK