summaryrefslogtreecommitdiff
path: root/fs
diff options
context:
space:
mode:
authorMiklos Szeredi <mszeredi@suse.cz>2015-07-01 17:26:08 +0300
committerMiklos Szeredi <mszeredi@suse.cz>2015-07-01 17:26:08 +0300
commitee314a870e402f4e63b8a651bc96c740ed69cb31 (patch)
tree3fa8edecd9539588427df90ab8317a48d2209a4f /fs
parent46c34a348b0ac74fc9646de8ef0c5b8b29f09da9 (diff)
downloadlinux-ee314a870e402f4e63b8a651bc96c740ed69cb31.tar.xz
fuse: abort: no fc->lock needed for request ending
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>
Diffstat (limited to 'fs')
-rw-r--r--fs/fuse/dev.c14
1 files changed, 5 insertions, 9 deletions
diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index fa53e5e9261a..dcfc87172a47 100644
--- a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c
@@ -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);