diff options
| author |   Trond Myklebust <trond.myklebust@hammerspace.com> | 2021-04-12 12:15:50 -0400 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@hammerspace.com> | 2021-04-13 10:04:05 -0400 |
| commit | 993e2d4bd9efc41f0943de39e9374ffdfde62e87 (patch) | |
| tree | 2bb64ddf7fec852900867bb2a1ebd7390d11dfad /fs/nfs/write.c | |
| parent | NFSv4: Fix value of decode_fsinfo_maxsz (diff) | |
| download | linux-dev-993e2d4bd9efc41f0943de39e9374ffdfde62e87.tar.xz linux-dev-993e2d4bd9efc41f0943de39e9374ffdfde62e87.zip | |
NFSv4: Don't modify the change attribute cached in the inode
When the client is caching data and a write delegation is held, then the
server may send a CB_GETATTR to query the attributes. When this happens,
the client is supposed to bump the change attribute value that it
returns if it holds cached data.
However that process uses a value that is stored in the delegation. We
do not want to bump the change attribute held in the inode.
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Diffstat (limited to 'fs/nfs/write.c')
| -rw-r--r-- | fs/nfs/write.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 61d1174935b6..3bf82178166a 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c @@ -764,9 +764,6 @@ static void nfs_inode_add_request(struct inode *inode, struct nfs_page *req) * with invalidate/truncate. */ spin_lock(&mapping->private_lock); - if (!nfs_have_writebacks(inode) && - NFS_PROTO(inode)->have_delegation(inode, FMODE_WRITE)) - inode_inc_iversion_raw(inode); if (likely(!PageSwapCache(req->wb_page))) { set_bit(PG_MAPPED, &req->wb_flags); SetPagePrivate(req->wb_page); |