summaryrefslogtreecommitdiffstats
path: root/fs/open.c
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2014-03-14 10:40:46 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2014-04-01 23:19:12 -0400
commit0ccb286346c4c0644be17f04a9eb23ad99262882 (patch)
tree2aa7197509c0d3c3f63db7fd0be151a0d0475ac1 /fs/open.c
parent4597e695b8baa3e2620da89c7593be70cf20566b (diff)
downloadlinux-0ccb286346c4c0644be17f04a9eb23ad99262882.tar.bz2
fold __get_file_write_access() into its only caller
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/open.c')
-rw-r--r--fs/open.c25
1 files changed, 6 insertions, 19 deletions
diff --git a/fs/open.c b/fs/open.c
index 8d0b6adfe7b8..ebef0c5fa10c 100644
--- a/fs/open.c
+++ b/fs/open.c
@@ -632,24 +632,6 @@ out:
return error;
}
-/*
- * You have to be very careful that these write
- * counts get cleaned up in error cases and
- * upon __fput(). This should probably never
- * be called outside of __dentry_open().
- */
-static inline int __get_file_write_access(struct inode *inode,
- struct vfsmount *mnt)
-{
- int error = get_write_access(inode);
- if (error)
- return error;
- error = __mnt_want_write(mnt);
- if (error)
- put_write_access(inode);
- return error;
-}
-
int open_check_o_direct(struct file *f)
{
/* NB: we're sure to have correct a_ops only after f_op->open */
@@ -680,9 +662,14 @@ static int do_dentry_open(struct file *f,
path_get(&f->f_path);
inode = f->f_inode = f->f_path.dentry->d_inode;
if (f->f_mode & FMODE_WRITE && !special_file(inode->i_mode)) {
- error = __get_file_write_access(inode, f->f_path.mnt);
+ error = get_write_access(inode);
if (error)
goto cleanup_file;
+ error = __mnt_want_write(f->f_path.mnt);
+ if (error) {
+ put_write_access(inode);
+ goto cleanup_file;
+ }
}
f->f_mapping = inode->i_mapping;