nfs/write: Use common error handling code in nfs_lock_and_join_requests()
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 7 Nov 2017 07:51:00 +0000 (08:51 +0100)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Fri, 17 Nov 2017 21:43:50 +0000 (16:43 -0500)
Add a jump target so that a bit of exception handling can be better reused
at the end of this function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
fs/nfs/write.c

index babebbc..5b5f464 100644 (file)
@@ -487,10 +487,8 @@ try_again:
        }
 
        ret = nfs_page_group_lock(head);
-       if (ret < 0) {
-               nfs_unlock_and_release_request(head);
-               return ERR_PTR(ret);
-       }
+       if (ret < 0)
+               goto release_request;
 
        /* lock each request in the page group */
        total_bytes = head->wb_bytes;
@@ -515,8 +513,7 @@ try_again:
                        if (ret < 0) {
                                nfs_unroll_locks(inode, head, subreq);
                                nfs_release_request(subreq);
-                               nfs_unlock_and_release_request(head);
-                               return ERR_PTR(ret);
+                               goto release_request;
                        }
                }
                /*
@@ -532,8 +529,8 @@ try_again:
                        nfs_page_group_unlock(head);
                        nfs_unroll_locks(inode, head, subreq);
                        nfs_unlock_and_release_request(subreq);
-                       nfs_unlock_and_release_request(head);
-                       return ERR_PTR(-EIO);
+                       ret = -EIO;
+                       goto release_request;
                }
        }
 
@@ -576,6 +573,10 @@ try_again:
        /* still holds ref on head from nfs_page_find_head_request
         * and still has lock on head from lock loop */
        return head;
+
+release_request:
+       nfs_unlock_and_release_request(head);
+       return ERR_PTR(ret);
 }
 
 static void nfs_write_error_remove_page(struct nfs_page *req)