diff options
| author | Al Viro <viro@ZenIV.linux.org.uk> | 2014-11-19 20:35:58 +0100 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2014-11-20 12:18:08 +0100 |
| commit | ec7d879c457611e540cb465c25f3040facbd1185 (patch) | |
| tree | 32412d1a4ad9d8408eeceb12c68321ae9a4c1f41 /fs | |
| parent | GFS2: gfs2_dir_get_hash_table(): avoiding deferred vfree() is easy here... (diff) | |
| download | linux-ec7d879c457611e540cb465c25f3040facbd1185.tar.xz linux-ec7d879c457611e540cb465c25f3040facbd1185.zip | |
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')
| -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, if (d != NULL) dentry = d; if (dentry->d_inode) { - if (!(*opened & FILE_OPENED)) { - if (d == NULL) - dget(dentry); - return finish_no_open(file, dentry); - } + if (!(*opened & FILE_OPENED)) + return finish_no_open(file, d); dput(d); return 0; } |