From: Miklos Szeredi Move dput/mntput pair from request_end() to fuse_release_end(), because there's no other place they are used. Signed-off-by: Miklos Szeredi Signed-off-by: Andrew Morton --- fs/fuse/dev.c | 2 -- fs/fuse/file.c | 8 ++++++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff -puN fs/fuse/dev.c~fuse-cleanup-in-release fs/fuse/dev.c --- a/fs/fuse/dev.c~fuse-cleanup-in-release +++ a/fs/fuse/dev.c @@ -231,8 +231,6 @@ static void request_end(struct fuse_conn fc->num_background--; } spin_unlock(&fc->lock); - dput(req->dentry); - mntput(req->vfsmount); wake_up(&req->waitq); if (end) end(fc, req); diff -puN fs/fuse/file.c~fuse-cleanup-in-release fs/fuse/file.c --- a/fs/fuse/file.c~fuse-cleanup-in-release +++ a/fs/fuse/file.c @@ -71,11 +71,19 @@ static struct fuse_file *fuse_file_get(s return ff; } +static void fuse_release_end(struct fuse_conn *fc, struct fuse_req *req) +{ + dput(req->dentry); + mntput(req->vfsmount); + fuse_put_request(fc, req); +} + static void fuse_file_put(struct fuse_file *ff) { if (atomic_dec_and_test(&ff->count)) { struct fuse_req *req = ff->reserved_req; struct fuse_conn *fc = get_fuse_conn(req->dentry->d_inode); + req->end = fuse_release_end; request_send_background(fc, req); kfree(ff); } _