diff options
| author | Chuck Lever <[email protected]> | 2006-09-20 18:33:04 +0000 |
|---|---|---|
| committer | Trond Myklebust <[email protected]> | 2006-09-23 03:25:05 +0000 |
| commit | f551e44ff11d3e2ec8f37907bb88ec2433cc8b74 (patch) | |
| tree | f86527c24391f70d72a28111df5c541959e8f293 /fs/nfs/write.c | |
| parent | NFSv4: rpc_mkpipe creating socket inodes w/out sk buffers (diff) | |
| download | kernel-f551e44ff11d3e2ec8f37907bb88ec2433cc8b74.tar.gz kernel-f551e44ff11d3e2ec8f37907bb88ec2433cc8b74.zip | |
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 <[email protected]>
Signed-off-by: Trond Myklebust <[email protected]>
Diffstat (limited to 'fs/nfs/write.c')
| -rw-r--r-- | fs/nfs/write.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 38ba5c09af08..c12effb46fe5 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c @@ -1253,7 +1253,13 @@ int nfs_writeback_done(struct rpc_task *task, struct nfs_write_data *data) dprintk("NFS: %4d nfs_writeback_done (status %d)\n", task->tk_pid, task->tk_status); - /* Call the NFS version-specific code */ + /* + * ->write_done will attempt to use post-op attributes to detect + * conflicting writes by other clients. A strict interpretation + * of close-to-open would allow us to continue caching even if + * another writer had changed the file, but some applications + * depend on tighter cache coherency when writing. + */ status = NFS_PROTO(data->inode)->write_done(task, data); if (status != 0) return status; |