nfs: change nfs_page_group_lock argument
authorWeston Andros Adamson <dros@primarydata.com>
Fri, 8 Aug 2014 15:00:53 +0000 (11:00 -0400)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Fri, 22 Aug 2014 22:04:42 +0000 (18:04 -0400)
Flip the meaning of the second argument from 'wait' to 'nonblock' to
match related functions. Update all five calls to reflect this change.

Signed-off-by: Weston Andros Adamson <dros@primarydata.com>
Reviewed-by: Peng Tao <tao.peng@primarydata.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
fs/nfs/pagelist.c
fs/nfs/write.c

index ba491926df5f7df2db1e224c96e7e070bd36dcec..7efa61586bc46f51cd4e683867b1e9d016022583 100644 (file)
@@ -139,13 +139,14 @@ nfs_iocounter_wait(struct nfs_io_counter *c)
 /*
  * nfs_page_group_lock - lock the head of the page group
  * @req - request in group that is to be locked
+ * @nonblock - if true don't block waiting for lock
  *
  * this lock must be held if modifying the page group list
  *
  * returns result from wait_on_bit_lock: 0 on success, < 0 on error
  */
 int
-nfs_page_group_lock(struct nfs_page *req, bool wait)
+nfs_page_group_lock(struct nfs_page *req, bool nonblock)
 {
        struct nfs_page *head = req->wb_head;
        int ret;
@@ -155,7 +156,7 @@ nfs_page_group_lock(struct nfs_page *req, bool wait)
        do {
                ret = wait_on_bit_lock(&head->wb_flags, PG_HEADLOCK,
                        TASK_UNINTERRUPTIBLE);
-       } while (wait && ret != 0);
+       } while (!nonblock && ret != 0);
 
        WARN_ON_ONCE(ret > 0);
        return ret;
@@ -219,7 +220,7 @@ bool nfs_page_group_sync_on_bit(struct nfs_page *req, unsigned int bit)
 {
        bool ret;
 
-       nfs_page_group_lock(req, true);
+       nfs_page_group_lock(req, false);
        ret = nfs_page_group_sync_on_bit_locked(req, bit);
        nfs_page_group_unlock(req);
 
@@ -860,7 +861,7 @@ static int __nfs_pageio_add_request(struct nfs_pageio_descriptor *desc,
        unsigned int offset, pgbase;
        int ret;
 
-       ret = nfs_page_group_lock(req, false);
+       ret = nfs_page_group_lock(req, true);
        if (ret < 0) {
                desc->pg_error = ret;
                return 0;
@@ -886,7 +887,7 @@ static int __nfs_pageio_add_request(struct nfs_pageio_descriptor *desc,
                        if (desc->pg_recoalesce)
                                return 0;
                        /* retry add_request for this subreq */
-                       ret = nfs_page_group_lock(req, false);
+                       ret = nfs_page_group_lock(req, true);
                        if (ret < 0) {
                                desc->pg_error = ret;
                                return 0;
index e3b5cf28bdc5c2dbfba5d3c16b06b5724afe6c60..27715306f24bbf3f400bfeee7376a2e929fb214d 100644 (file)
@@ -241,7 +241,7 @@ static bool nfs_page_group_covers_page(struct nfs_page *req)
        unsigned int pos = 0;
        unsigned int len = nfs_page_length(req->wb_page);
 
-       nfs_page_group_lock(req, true);
+       nfs_page_group_lock(req, false);
 
        do {
                tmp = nfs_page_group_search_locked(req->wb_head, pos);
@@ -479,7 +479,7 @@ try_again:
        }
 
        /* lock each request in the page group */
-       ret = nfs_page_group_lock(head, false);
+       ret = nfs_page_group_lock(head, true);
        if (ret < 0)
                return ERR_PTR(ret);
        subreq = head;