diff options
author | Trond Myklebust <trond.myklebust@primarydata.com> | 2016-12-19 09:47:32 -0500 |
---|---|---|
committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2016-12-19 17:29:56 -0500 |
commit | a5f925bccce7b0dc083f0c5a8652600881cc38ab (patch) | |
tree | c3a0496e93c9abfaa0b59313aa8753f74035ef69 | |
parent | e71708d4df1d4b81427badb9ac4bc4a813338b17 (diff) |
NFS: Don't abuse NFS_INO_REVAL_FORCED in nfs_post_op_update_inode_locked()
The NFS_INO_REVAL_FORCED flag now really only has meaning for the
case when we've just been handed a delegation for a file that was already
cached, and we're unsure about that cache.
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
-rw-r--r-- | fs/nfs/inode.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/nfs/inode.c b/fs/nfs/inode.c index a0cd79646c1b..5561d78b7998 100644 --- a/fs/nfs/inode.c +++ b/fs/nfs/inode.c | |||
@@ -1550,13 +1550,6 @@ static int nfs_post_op_update_inode_locked(struct inode *inode, struct nfs_fattr | |||
1550 | { | 1550 | { |
1551 | unsigned long invalid = NFS_INO_INVALID_ATTR; | 1551 | unsigned long invalid = NFS_INO_INVALID_ATTR; |
1552 | 1552 | ||
1553 | /* | ||
1554 | * Don't revalidate the pagecache if we hold a delegation, but do | ||
1555 | * force an attribute update | ||
1556 | */ | ||
1557 | if (NFS_PROTO(inode)->have_delegation(inode, FMODE_READ)) | ||
1558 | invalid = NFS_INO_INVALID_ATTR|NFS_INO_REVAL_FORCED; | ||
1559 | |||
1560 | if (S_ISDIR(inode->i_mode)) | 1553 | if (S_ISDIR(inode->i_mode)) |
1561 | invalid |= NFS_INO_INVALID_DATA; | 1554 | invalid |= NFS_INO_INVALID_DATA; |
1562 | nfs_set_cache_invalid(inode, invalid); | 1555 | nfs_set_cache_invalid(inode, invalid); |