Commit 222f1d69 authored Jan 16, 2006 by Miklos Szeredi Committed by Linus Torvalds Jan 16, 2006

[PATCH] fuse: fuse_copy_finish() order fix



fuse_copy_finish() must be called before request_end(), since the later might
sleep, and no sleeping is allowed between fuse_copy_one() and
fuse_copy_finish() because of kmap_atomic()/kunmap_atomic() used in them.

Signed-off-by: Miklos Szeredi <miklos@szeredi.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

parent f87fd4c2

fs/fuse/dev.c

+3 −1

Original line number	Diff line number	Diff line
		@@ -773,8 +773,10 @@ static ssize_t fuse_dev_writev(struct file file, const struct iovec iov,

		list_del_init(&req->list);
		if (req->interrupted) {
		request_end(fc, req);
		spin_unlock(&fuse_lock);
		fuse_copy_finish(&cs);
		spin_lock(&fuse_lock);
		request_end(fc, req);
		return -ENOENT;
		}
		req->out.h = oh;