NFS: Create a common multiple_pgios() function
authorAnna Schumaker <Anna.Schumaker@netapp.com>
Tue, 6 May 2014 13:12:38 +0000 (09:12 -0400)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Thu, 29 May 2014 15:11:40 +0000 (11:11 -0400)
Once again, these two functions look identical in the read and write
case.  Time to combine them together!

Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/internal.h
fs/nfs/pagelist.c
fs/nfs/read.c
fs/nfs/write.c

index be4f2a7e91788461c140d71980547a3addf3638f..886273a8916b0037b45d322aee6b938556aa030a 100644 (file)
@@ -243,6 +243,7 @@ void nfs_pgio_data_release(struct nfs_pgio_data *);
 int nfs_generic_pgio(struct nfs_pageio_descriptor *, struct nfs_pgio_header *);
 int nfs_initiate_pgio(struct rpc_clnt *, struct nfs_pgio_data *,
                      const struct rpc_call_ops *, int, int);
+int nfs_do_multiple_pgios(struct list_head *, const struct rpc_call_ops *, int);
 
 static inline void nfs_iocounter_init(struct nfs_io_counter *c)
 {
index ab5b1850ca4fcbbd916ac79d294d636a0c49e2c0..2471e0278811aa90592bbc2b73bb0274a99a53e8 100644 (file)
@@ -493,6 +493,27 @@ out:
 }
 EXPORT_SYMBOL_GPL(nfs_initiate_pgio);
 
+int nfs_do_multiple_pgios(struct list_head *head,
+                                const struct rpc_call_ops *call_ops,
+                                int how)
+{
+       struct nfs_pgio_data *data;
+       int ret = 0;
+
+       while (!list_empty(head)) {
+               int ret2;
+
+               data = list_first_entry(head, struct nfs_pgio_data, list);
+               list_del_init(&data->list);
+
+               ret2 = nfs_initiate_pgio(NFS_CLIENT(data->header->inode),
+                                        data, call_ops, how, 0);
+               if (ret == 0)
+                        ret = ret2;
+       }
+       return ret;
+}
+
 /**
  * nfs_pgio_error - Clean up from a pageio error
  * @desc: IO descriptor
index 0359b0d76ef67f9232791d37e6264195e5a8bf3d..b834d458a2b6d8906f4a09492565de0f0df547c0 100644 (file)
@@ -161,34 +161,6 @@ static void nfs_initiate_read(struct nfs_pgio_data *data, struct rpc_message *ms
        NFS_PROTO(inode)->read_setup(data, msg);
 }
 
-static int nfs_do_read(struct nfs_pgio_data *data,
-               const struct rpc_call_ops *call_ops)
-{
-       struct inode *inode = data->header->inode;
-
-       return nfs_initiate_pgio(NFS_CLIENT(inode), data, call_ops, 0, 0);
-}
-
-static int
-nfs_do_multiple_reads(struct list_head *head,
-               const struct rpc_call_ops *call_ops)
-{
-       struct nfs_pgio_data *data;
-       int ret = 0;
-
-       while (!list_empty(head)) {
-               int ret2;
-
-               data = list_first_entry(head, struct nfs_pgio_data, list);
-               list_del_init(&data->list);
-
-               ret2 = nfs_do_read(data, call_ops);
-               if (ret == 0)
-                       ret = ret2;
-       }
-       return ret;
-}
-
 static void
 nfs_async_read_error(struct list_head *head)
 {
@@ -222,8 +194,8 @@ static int nfs_generic_pg_readpages(struct nfs_pageio_descriptor *desc)
        atomic_inc(&hdr->refcnt);
        ret = nfs_generic_pgio(desc, hdr);
        if (ret == 0)
-               ret = nfs_do_multiple_reads(&hdr->rpc_list,
-                                           desc->pg_rpc_callops);
+               ret = nfs_do_multiple_pgios(&hdr->rpc_list,
+                                           desc->pg_rpc_callops, 0);
        if (atomic_dec_and_test(&hdr->refcnt))
                hdr->completion_ops->completion(hdr);
        return ret;
index e46a1fc6c1fe3f8285bcb24ccf648ca53a80abbc..d3fa181053add276e608eb4154942ee5fd8ebacf 100644 (file)
@@ -945,35 +945,6 @@ static void nfs_initiate_write(struct nfs_pgio_data *data, struct rpc_message *m
                                 &task_setup_data->rpc_client, msg, data);
 }
 
-static int nfs_do_write(struct nfs_pgio_data *data,
-               const struct rpc_call_ops *call_ops,
-               int how)
-{
-       struct inode *inode = data->header->inode;
-
-       return nfs_initiate_pgio(NFS_CLIENT(inode), data, call_ops, how, 0);
-}
-
-static int nfs_do_multiple_writes(struct list_head *head,
-               const struct rpc_call_ops *call_ops,
-               int how)
-{
-       struct nfs_pgio_data *data;
-       int ret = 0;
-
-       while (!list_empty(head)) {
-               int ret2;
-
-               data = list_first_entry(head, struct nfs_pgio_data, list);
-               list_del_init(&data->list);
-               
-               ret2 = nfs_do_write(data, call_ops, how);
-                if (ret == 0)
-                        ret = ret2;
-       }
-       return ret;
-}
-
 /* If a nfs_flush_* function fails, it should remove reqs from @head and
  * call this on each, which will prepare them to be retried on next
  * writeback using standard nfs.
@@ -1018,7 +989,7 @@ static int nfs_generic_pg_writepages(struct nfs_pageio_descriptor *desc)
        atomic_inc(&hdr->refcnt);
        ret = nfs_generic_pgio(desc, hdr);
        if (ret == 0)
-               ret = nfs_do_multiple_writes(&hdr->rpc_list,
+               ret = nfs_do_multiple_pgios(&hdr->rpc_list,
                                             desc->pg_rpc_callops,
                                             desc->pg_ioflags);
        if (atomic_dec_and_test(&hdr->refcnt))