diff options
author | Wendy Cheng <wcheng@redhat.com> | 2007-06-27 17:07:08 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-07-09 03:24:08 -0400 |
commit | bb9bcf061660661c57ddcf31337529f82414b937 (patch) | |
tree | 0876874e5252c4939b8e7bbd62a22a6eb4ad1abf /fs/gfs2/ops_export.c | |
parent | f4fadb23ca49abd2f1387a0b7e78b385ebc760ce (diff) |
[GFS2] Obtaining no_formal_ino from directory entry
GFS2 lookup code doesn't ask for inode shared glock. This implies during
in-memory inode creation for existing file, GFS2 will not disk-read in
the inode contents. This leaves no_formal_ino un-initialized during
lookup time. The un-initialized no_formal_ino is subsequently encoded
into file handle. Clients will get ESTALE error whenever it tries to
access these files.
Signed-off-by: S. Wendy Cheng <wcheng@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/ops_export.c')
-rw-r--r-- | fs/gfs2/ops_export.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/gfs2/ops_export.c b/fs/gfs2/ops_export.c index d07230ee5fc0..0fe14478a54d 100644 --- a/fs/gfs2/ops_export.c +++ b/fs/gfs2/ops_export.c | |||
@@ -245,7 +245,9 @@ static struct dentry *gfs2_get_dentry(struct super_block *sb, void *inum_obj) | |||
245 | gfs2_glock_dq_uninit(&rgd_gh); | 245 | gfs2_glock_dq_uninit(&rgd_gh); |
246 | gfs2_glock_dq_uninit(&ri_gh); | 246 | gfs2_glock_dq_uninit(&ri_gh); |
247 | 247 | ||
248 | inode = gfs2_inode_lookup(sb, inum->no_addr, fh_obj->imode); | 248 | inode = gfs2_inode_lookup(sb, fh_obj->imode, |
249 | inum->no_addr, | ||
250 | inum->no_formal_ino); | ||
249 | if (!inode) | 251 | if (!inode) |
250 | goto fail; | 252 | goto fail; |
251 | if (IS_ERR(inode)) { | 253 | if (IS_ERR(inode)) { |