diff options
| author | Wendy Cheng <wcheng@redhat.com> | 2007-06-27 17:07:08 -0400 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-07-09 08:24:08 +0100 |
| commit | bb9bcf061660661c57ddcf31337529f82414b937 (patch) | |
| tree | 0876874e5252c4939b8e7bbd62a22a6eb4ad1abf /fs/gfs2/ops_export.c | |
| parent | f4fadb23ca49abd2f1387a0b7e78b385ebc760ce (diff) | |
| download | op-kernel-dev-bb9bcf061660661c57ddcf31337529f82414b937.zip op-kernel-dev-bb9bcf061660661c57ddcf31337529f82414b937.tar.gz | |
[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 d07230e..0fe1447 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) gfs2_glock_dq_uninit(&rgd_gh); gfs2_glock_dq_uninit(&ri_gh); - inode = gfs2_inode_lookup(sb, inum->no_addr, fh_obj->imode); + inode = gfs2_inode_lookup(sb, fh_obj->imode, + inum->no_addr, + inum->no_formal_ino); if (!inode) goto fail; if (IS_ERR(inode)) { |
