NFS: Remove an incorrect revalidation in nfs4_update_changeattr_locked()
authorTrond Myklebust <trond.myklebust@hammerspace.com>
Tue, 8 Feb 2022 17:14:44 +0000 (12:14 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 23 Feb 2022 11:03:15 +0000 (12:03 +0100)
commitf2238b4e83b60c243c35bc352161c8ce7e790725
tree8a93bc42c212846965fa747cd6576e3e6133063c
parent4b9c861a589e3486fb3806c96953d27853401d04
NFS: Remove an incorrect revalidation in nfs4_update_changeattr_locked()

commit 9d047bf68fe8cdb4086deaf4edd119731a9481ed upstream.

In nfs4_update_changeattr_locked(), we don't need to set the
NFS_INO_REVAL_PAGECACHE flag, because we already know the value of the
change attribute, and we're already flagging the size. In fact, this
forces us to revalidate the change attribute a second time for no good
reason.
This extra flag appears to have been introduced as part of the xattr
feature, when update_changeattr_locked() was converted for use by the
xattr code.

Fixes: 1b523ca972ed ("nfs: modify update_changeattr to deal with regular files")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/nfs/nfs4proc.c