diff options
author | Al Viro <viro@ZenIV.linux.org.uk> | 2014-11-19 14:35:58 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2014-11-20 06:18:08 -0500 |
commit | ec7d879c457611e540cb465c25f3040facbd1185 (patch) | |
tree | 32412d1a4ad9d8408eeceb12c68321ae9a4c1f41 /fs/gfs2 | |
parent | 9265f1d0c7593e3e7e1e94a4a83a6dea34230a35 (diff) |
GFS2: gfs2_atomic_open(): simplify the use of finish_no_open()
In ->atomic_open(inode, dentry, file, opened) calling finish_no_open(file, NULL)
is equivalent to dget(dentry); return finish_no_open(file, dentry);
No need to open-code that...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
-rw-r--r-- | fs/gfs2/inode.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index 9e8545bbc55c..9054002ebe70 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c | |||
@@ -1245,11 +1245,8 @@ static int gfs2_atomic_open(struct inode *dir, struct dentry *dentry, | |||
1245 | if (d != NULL) | 1245 | if (d != NULL) |
1246 | dentry = d; | 1246 | dentry = d; |
1247 | if (dentry->d_inode) { | 1247 | if (dentry->d_inode) { |
1248 | if (!(*opened & FILE_OPENED)) { | 1248 | if (!(*opened & FILE_OPENED)) |
1249 | if (d == NULL) | 1249 | return finish_no_open(file, d); |
1250 | dget(dentry); | ||
1251 | return finish_no_open(file, dentry); | ||
1252 | } | ||
1253 | dput(d); | 1250 | dput(d); |
1254 | return 0; | 1251 | return 0; |
1255 | } | 1252 | } |