diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2006-09-20 14:33:04 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-09-22 23:25:05 -0400 |
| commit | f551e44ff11d3e2ec8f37907bb88ec2433cc8b74 (patch) | |
| tree | f86527c24391f70d72a28111df5c541959e8f293 /fs/nfs/inode.c | |
| parent | a53a3c58fd83e572a7c768d88b4c4e9840a57e82 (diff) | |
| download | op-kernel-dev-f551e44ff11d3e2ec8f37907bb88ec2433cc8b74.zip op-kernel-dev-f551e44ff11d3e2ec8f37907bb88ec2433cc8b74.tar.gz | |
NFS: add comments clarifying the use of nfs_post_op_update()
Comments-only change to clarify a detail of the NFS protocol and how it is
implemented in Linux.
Test plan:
None.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/inode.c')
| -rw-r--r-- | fs/nfs/inode.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/nfs/inode.c b/fs/nfs/inode.c index a56add0..e8c143d 100644 --- a/fs/nfs/inode.c +++ b/fs/nfs/inode.c @@ -840,6 +840,12 @@ int nfs_refresh_inode(struct inode *inode, struct nfs_fattr *fattr) * * After an operation that has changed the inode metadata, mark the * attribute cache as being invalid, then try to update it. + * + * NB: if the server didn't return any post op attributes, this + * function will force the retrieval of attributes before the next + * NFS request. Thus it should be used only for operations that + * are expected to change one or more attributes, to avoid + * unnecessary NFS requests and trips through nfs_update_inode(). */ int nfs_post_op_update_inode(struct inode *inode, struct nfs_fattr *fattr) { |
