diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-03-20 13:44:36 -0500 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-03-20 13:44:36 -0500 |
commit | fad61490419b3e494f300e9b2579810ef3bcda31 (patch) | |
tree | 1bca68921a83d4dc0219aa8d46c3c8e8313965c2 /include/linux/nfs_fs.h | |
parent | e17b1fc4b35399935f00a635206e183d9292fe4f (diff) |
nfs: Use UNSTABLE + COMMIT for NFS O_DIRECT writes
Currently NFS O_DIRECT writes use FILE_SYNC so that a COMMIT is not
necessary. This simplifies the internal logic, but this could be a
difficult workload for some servers.
Instead, let's send UNSTABLE writes, and after they all complete, send a
COMMIT for the dirty range. After the COMMIT returns successfully, then do
the wake_up or fire off aio_complete().
Test plan:
Async direct I/O tests against Solaris (or any server that requires
committed unstable writes). Reboot server during test.
Based on an earlier patch by Chuck Lever <cel@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'include/linux/nfs_fs.h')
-rw-r--r-- | include/linux/nfs_fs.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/nfs_fs.h b/include/linux/nfs_fs.h index 423f202b881c..9f84c8a5ea43 100644 --- a/include/linux/nfs_fs.h +++ b/include/linux/nfs_fs.h | |||
@@ -422,6 +422,7 @@ void nfs_commit_free(struct nfs_write_data *p); | |||
422 | extern int nfs_sync_inode(struct inode *, unsigned long, unsigned int, int); | 422 | extern int nfs_sync_inode(struct inode *, unsigned long, unsigned int, int); |
423 | #if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) | 423 | #if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) |
424 | extern int nfs_commit_inode(struct inode *, int); | 424 | extern int nfs_commit_inode(struct inode *, int); |
425 | extern void nfs_commit_release(void *wdata); | ||
425 | #else | 426 | #else |
426 | static inline int | 427 | static inline int |
427 | nfs_commit_inode(struct inode *inode, int how) | 428 | nfs_commit_inode(struct inode *inode, int how) |