fuse: no fc->lock in request_end()
authorMiklos Szeredi <mszeredi@suse.cz>
Wed, 1 Jul 2015 14:26:07 +0000 (16:26 +0200)
committerMiklos Szeredi <mszeredi@suse.cz>
Wed, 1 Jul 2015 14:26:07 +0000 (16:26 +0200)
No longer need to call request_end() with the connection lock held.  We
still protect the background counters and queue with fc->lock, so acquire
it if necessary.

Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Reviewed-by: Ashish Samant <ashish.samant@oracle.com>
fs/fuse/dev.c

index 4e1144a384389576da85cf4ad8d5d18da8bc4f80..f2c7dd90ebfcd2255695bf4c513548a2779961c6 100644 (file)
@@ -376,18 +376,13 @@ static void flush_bg_queue(struct fuse_conn *fc)
  * was closed.  The requester thread is woken up (if still waiting),
  * the 'end' callback is called if given, else the reference to the
  * request is released
- *
- * Called with fc->lock, unlocks it
  */
 static void request_end(struct fuse_conn *fc, struct fuse_req *req)
-__releases(fc->lock)
 {
        struct fuse_iqueue *fiq = &fc->iq;
 
-       if (test_and_set_bit(FR_FINISHED, &req->flags)) {
-               spin_unlock(&fc->lock);
+       if (test_and_set_bit(FR_FINISHED, &req->flags))
                return;
-       }
 
        spin_lock(&fiq->waitq.lock);
        list_del_init(&req->intr_entry);
@@ -395,6 +390,7 @@ __releases(fc->lock)
        WARN_ON(test_bit(FR_PENDING, &req->flags));
        WARN_ON(test_bit(FR_SENT, &req->flags));
        if (test_bit(FR_BACKGROUND, &req->flags)) {
+               spin_lock(&fc->lock);
                clear_bit(FR_BACKGROUND, &req->flags);
                if (fc->num_background == fc->max_background)
                        fc->blocked = 0;
@@ -411,8 +407,8 @@ __releases(fc->lock)
                fc->num_background--;
                fc->active_background--;
                flush_bg_queue(fc);
+               spin_unlock(&fc->lock);
        }
-       spin_unlock(&fc->lock);
        wake_up(&req->waitq);
        if (req->end)
                req->end(fc, req);
@@ -1290,6 +1286,7 @@ static ssize_t fuse_dev_do_read(struct fuse_conn *fc, struct file *file,
                /* SETXATTR is special, since it may contain too large data */
                if (in->h.opcode == FUSE_SETXATTR)
                        req->out.h.error = -E2BIG;
+               spin_unlock(&fc->lock);
                request_end(fc, req);
                goto restart;
        }
@@ -1333,6 +1330,7 @@ out_end:
        if (!test_bit(FR_PRIVATE, &req->flags))
                list_del_init(&req->list);
        spin_unlock(&fpq->lock);
+       spin_unlock(&fc->lock);
        request_end(fc, req);
        return err;
 
@@ -1951,6 +1949,7 @@ static ssize_t fuse_dev_do_write(struct fuse_conn *fc,
        if (!test_bit(FR_PRIVATE, &req->flags))
                list_del_init(&req->list);
        spin_unlock(&fpq->lock);
+       spin_unlock(&fc->lock);
        request_end(fc, req);
 
        return err ? err : nbytes;
@@ -2095,6 +2094,7 @@ __acquires(fc->lock)
                clear_bit(FR_PENDING, &req->flags);
                clear_bit(FR_SENT, &req->flags);
                list_del_init(&req->list);
+               spin_unlock(&fc->lock);
                request_end(fc, req);
                spin_lock(&fc->lock);
        }
@@ -2177,6 +2177,7 @@ void fuse_abort_conn(struct fuse_conn *fc)
                        req = list_first_entry(&to_end1, struct fuse_req, list);
                        __fuse_get_request(req);
                        list_del_init(&req->list);
+                       spin_unlock(&fc->lock);
                        request_end(fc, req);
                        spin_lock(&fc->lock);
                }