diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2010-09-23 09:00:31 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2010-09-23 09:00:31 -0400 |
commit | c2048b003cfb840ad81bdc6eb55beb12a19a222e (patch) | |
tree | 17b3b09141cfb1c2ca13759bac365ef5050e5dc9 /fs/gfs2/dentry.c | |
parent | f57a024ed2ee00d62515ffd4040f5ded3e0eb853 (diff) |
GFS2: Remove localcaching mount option
This option defaulted to on for lock_nolock mounts and off
otherwise. The only function was to avoid the revalidation of
dentries. In the cluster case, that is entirely pointless and
liable to cause coherency problems.
The patch changes the revalidation to depend upon whether the
fs is a local or cluster fs (i.e. it follows the existing default
behaviour). I very much doubt anybody ever used this option as
there is no reason to. Even so we will continue to accept it
on the mount command line, but ignore it.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/dentry.c')
-rw-r--r-- | fs/gfs2/dentry.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/dentry.c b/fs/gfs2/dentry.c index bb7907bde3d8..6798755b3858 100644 --- a/fs/gfs2/dentry.c +++ b/fs/gfs2/dentry.c | |||
@@ -49,7 +49,7 @@ static int gfs2_drevalidate(struct dentry *dentry, struct nameidata *nd) | |||
49 | ip = GFS2_I(inode); | 49 | ip = GFS2_I(inode); |
50 | } | 50 | } |
51 | 51 | ||
52 | if (sdp->sd_args.ar_localcaching) | 52 | if (sdp->sd_lockstruct.ls_ops->lm_mount == NULL) |
53 | goto valid; | 53 | goto valid; |
54 | 54 | ||
55 | had_lock = (gfs2_glock_is_locked_by_me(dip->i_gl) != NULL); | 55 | had_lock = (gfs2_glock_is_locked_by_me(dip->i_gl) != NULL); |