NFS: Simplify the nfs_read_completion functions

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Fred Isaman <iisaman@netapp.com>

diff --git a/fs/nfs/direct.c b/fs/nfs/direct.c
index f17e469..aab3016 100644 (file)
--- a/fs/nfs/direct.c
+++ b/fs/nfs/direct.c
@@ -243,36 +243,28 @@ static void nfs_direct_read_completion(struct nfs_pgio_header *hdr)
                dreq->count += hdr->good_bytes;
        spin_unlock(&dreq->lock);
 
-       if (!test_bit(NFS_IOHDR_ERROR, &hdr->flags)) {
-               while (!list_empty(&hdr->pages)) {
-                       struct nfs_page *req = nfs_list_entry(hdr->pages.next);
-                       struct page *page = req->wb_page;
-
-                       if (test_bit(NFS_IOHDR_EOF, &hdr->flags)) {
-                               if (bytes > hdr->good_bytes)
-                                       zero_user(page, 0, PAGE_SIZE);
-                               else if (hdr->good_bytes - bytes < PAGE_SIZE)
-                                       zero_user_segment(page,
-                                               hdr->good_bytes & ~PAGE_MASK,
-                                               PAGE_SIZE);
-                       }
-                       bytes += req->wb_bytes;
-                       nfs_list_remove_request(req);
-                       if (!PageCompound(page))
-                               set_page_dirty(page);
-                       nfs_direct_readpage_release(req);
+       while (!list_empty(&hdr->pages)) {
+               struct nfs_page *req = nfs_list_entry(hdr->pages.next);
+               struct page *page = req->wb_page;
+
+               if (test_bit(NFS_IOHDR_EOF, &hdr->flags)) {
+                       if (bytes > hdr->good_bytes)
+                               zero_user(page, 0, PAGE_SIZE);
+                       else if (hdr->good_bytes - bytes < PAGE_SIZE)
+                               zero_user_segment(page,
+                                       hdr->good_bytes & ~PAGE_MASK,
+                                       PAGE_SIZE);
                }
-       } else {
-               while (!list_empty(&hdr->pages)) {
-                       struct nfs_page *req = nfs_list_entry(hdr->pages.next);
-
-                       if (bytes < hdr->good_bytes)
-                               if (!PageCompound(req->wb_page))
-                                       set_page_dirty(req->wb_page);
-                       bytes += req->wb_bytes;
-                       nfs_list_remove_request(req);
-                       nfs_direct_readpage_release(req);
+               if (!PageCompound(page)) {
+                       if (test_bit(NFS_IOHDR_ERROR, &hdr->flags)) {
+                               if (bytes < hdr->good_bytes)
+                                       set_page_dirty(page);
+                       } else
+                               set_page_dirty(page);
                }
+               bytes += req->wb_bytes;
+               nfs_list_remove_request(req);
+               nfs_direct_readpage_release(req);
        }
 out_put:
        if (put_dreq(dreq))

diff --git a/fs/nfs/read.c b/fs/nfs/read.c
index 1961a19..37c9eb2 100644 (file)
--- a/fs/nfs/read.c
+++ b/fs/nfs/read.c
@@ -179,34 +179,26 @@ static void nfs_read_completion(struct nfs_pgio_header *hdr)
 
        if (test_bit(NFS_IOHDR_REDO, &hdr->flags))
                goto out;
-       if (!test_bit(NFS_IOHDR_ERROR, &hdr->flags)) {
-               while (!list_empty(&hdr->pages)) {
-                       struct nfs_page *req = nfs_list_entry(hdr->pages.next);
-                       struct page *page = req->wb_page;
-
-                       if (test_bit(NFS_IOHDR_EOF, &hdr->flags)) {
-                               if (bytes > hdr->good_bytes)
-                                       zero_user(page, 0, PAGE_SIZE);
-                               else if (hdr->good_bytes - bytes < PAGE_SIZE)
-                                       zero_user_segment(page,
-                                               hdr->good_bytes & ~PAGE_MASK,
-                                               PAGE_SIZE);
-                       }
-                       SetPageUptodate(page);
-                       nfs_list_remove_request(req);
-                       nfs_readpage_release(req);
-                       bytes += PAGE_SIZE;
+       while (!list_empty(&hdr->pages)) {
+               struct nfs_page *req = nfs_list_entry(hdr->pages.next);
+               struct page *page = req->wb_page;
+
+               if (test_bit(NFS_IOHDR_EOF, &hdr->flags)) {
+                       if (bytes > hdr->good_bytes)
+                               zero_user(page, 0, PAGE_SIZE);
+                       else if (hdr->good_bytes - bytes < PAGE_SIZE)
+                               zero_user_segment(page,
+                                       hdr->good_bytes & ~PAGE_MASK,
+                                       PAGE_SIZE);
                }
-       } else {
-               while (!list_empty(&hdr->pages)) {
-                       struct nfs_page *req = nfs_list_entry(hdr->pages.next);
-
-                       bytes += req->wb_bytes;
+               bytes += req->wb_bytes;
+               if (test_bit(NFS_IOHDR_ERROR, &hdr->flags)) {
                        if (bytes <= hdr->good_bytes)
-                               SetPageUptodate(req->wb_page);
-                       nfs_list_remove_request(req);
-                       nfs_readpage_release(req);
-               }
+                               SetPageUptodate(page);
+               } else
+                       SetPageUptodate(page);
+               nfs_list_remove_request(req);
+               nfs_readpage_release(req);
        }
 out:
        hdr->release(hdr);