NFS: Read cleanups
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 1 May 2012 15:21:43 +0000 (11:21 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 1 May 2012 17:48:13 +0000 (13:48 -0400)
Remove unused variables, and reformat some code.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Fred Isaman <iisaman@netapp.com>
fs/nfs/read.c

index 35e2dcebffe62af3d2d615e09bce67b70608650d..20a0293240605bdb2d5e8ac1e4eb7d6a755fa027 100644 (file)
@@ -341,8 +341,6 @@ static int nfs_pagein_multi(struct nfs_pageio_descriptor *desc,
        struct nfs_read_data *data;
        size_t rsize = desc->pg_bsize, nbytes;
        unsigned int offset;
-       int requests = 0;
-       int ret = 0;
 
        nfs_list_remove_request(req);
        nfs_list_add_request(req, &hdr->pages);
@@ -358,12 +356,11 @@ static int nfs_pagein_multi(struct nfs_pageio_descriptor *desc,
                data->pages.pagevec[0] = page;
                nfs_read_rpcsetup(data, len, offset);
                list_add(&data->list, &hdr->rpc_list);
-               requests++;
                nbytes -= len;
                offset += len;
-       } while(nbytes != 0);
+       } while (nbytes != 0);
        desc->pg_rpc_callops = &nfs_read_common_ops;
-       return ret;
+       return 0;
 out_bad:
        while (!list_empty(&hdr->rpc_list)) {
                data = list_first_entry(&hdr->rpc_list, struct nfs_read_data, list);
@@ -387,8 +384,7 @@ static int nfs_pagein_one(struct nfs_pageio_descriptor *desc,
                                                          desc->pg_count));
        if (!data) {
                desc->pg_completion_ops->error_cleanup(head);
-               ret = -ENOMEM;
-               goto out;
+               return -ENOMEM;
        }
 
        pages = data->pages.pagevec;
@@ -402,8 +398,7 @@ static int nfs_pagein_one(struct nfs_pageio_descriptor *desc,
        nfs_read_rpcsetup(data, desc->pg_count, 0);
        list_add(&data->list, &hdr->rpc_list);
        desc->pg_rpc_callops = &nfs_read_common_ops;
-out:
-       return ret;
+       return 0;
 }
 
 int nfs_generic_pagein(struct nfs_pageio_descriptor *desc,