fuse: abort: no fc->lock needed for request ending
authorMiklos Szeredi <mszeredi@suse.cz>
Wed, 1 Jul 2015 14:26:08 +0000 (16:26 +0200)
committerMiklos Szeredi <mszeredi@suse.cz>
Wed, 1 Jul 2015 14:26:08 +0000 (16:26 +0200)
In fuse_abort_conn() when all requests are on private lists we no longer
need fc->lock protection.

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

index fa53e5e9261a6b8a3ef6ca6cb6a4f577967a5f4b..dcfc87172a47a1daeef4d32682519cb95233285a 100644 (file)
@@ -2072,8 +2072,6 @@ static unsigned fuse_dev_poll(struct file *file, poll_table *wait)
  * This function releases and reacquires fc->lock
  */
 static void end_requests(struct fuse_conn *fc, struct list_head *head)
-__releases(fc->lock)
-__acquires(fc->lock)
 {
        while (!list_empty(head)) {
                struct fuse_req *req;
@@ -2082,9 +2080,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);
        }
 }
 
@@ -2160,20 +2156,20 @@ void fuse_abort_conn(struct fuse_conn *fc)
                wake_up_all_locked(&fiq->waitq);
                spin_unlock(&fiq->waitq.lock);
                kill_fasync(&fiq->fasync, SIGIO, POLL_IN);
+               end_polls(fc);
+               wake_up_all(&fc->blocked_waitq);
+               spin_unlock(&fc->lock);
 
                while (!list_empty(&to_end1)) {
                        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);
                }
                end_requests(fc, &to_end2);
-               end_polls(fc);
-               wake_up_all(&fc->blocked_waitq);
+       } else {
+               spin_unlock(&fc->lock);
        }
-       spin_unlock(&fc->lock);
 }
 EXPORT_SYMBOL_GPL(fuse_abort_conn);