From: Andy Adamson <andros@xxxxxxxxxx>
The LAYOUTCOMMIT call indicates an update to the file meta data is needed,
and should be called when clearing the I_DIRTY_SYNC state.
A call to LAYOUTCOMMIT in nfs_write_inode replaces the calls in nfs_wb_all,
nfs_commit_inode, and __nfs4_close.
Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
---
fs/nfs/nfs4state.c | 2 --
fs/nfs/write.c | 22 ++++++++++------------
2 files changed, 10 insertions(+), 14 deletions(-)
diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c
index d0dbdd4..e1207fa 100644
--- a/fs/nfs/nfs4state.c
+++ b/fs/nfs/nfs4state.c
@@ -589,8 +589,6 @@ static void __nfs4_close(struct path *path, struct nfs4_state *state, fmode_t fm
#ifdef CONFIG_NFS_V4_1
struct nfs_inode *nfsi = NFS_I(state->inode);
- if (layoutcommit_needed(nfsi))
- pnfs_layoutcommit_inode(state->inode, wait);
if (has_layout(nfsi) && nfsi->layout.roc_iomode) {
struct nfs4_pnfs_layout_segment range;
diff --git a/fs/nfs/write.c b/fs/nfs/write.c
index 0fd33cb..875d07f 100644
--- a/fs/nfs/write.c
+++ b/fs/nfs/write.c
@@ -1489,13 +1489,6 @@ static int nfs_commit_inode(struct inode *inode, int how)
wait_on_bit(&NFS_I(inode)->flags, NFS_INO_COMMIT,
nfs_wait_bit_killable,
TASK_KILLABLE);
-#ifdef CONFIG_NFS_V4_1
- if (may_wait && layoutcommit_needed(NFS_I(inode))) {
- error = pnfs_layoutcommit_inode(inode, 1);
- if (error < 0)
- return error;
- }
-#endif /* CONFIG_NFS_V4_1 */
} else
nfs_commit_clear_lock(NFS_I(inode));
out:
@@ -1545,7 +1538,16 @@ static int nfs_commit_unstable_pages(struct inode *inode, struct writeback_contr
int nfs_write_inode(struct inode *inode, struct writeback_control *wbc)
{
- return nfs_commit_unstable_pages(inode, wbc);
+ int ret;
+ ret = nfs_commit_unstable_pages(inode, wbc);
+#ifdef CONFIG_NFS_V4_1
+ if (ret >= 0 && layoutcommit_needed(NFS_I(inode))) {
+ int err = pnfs_layoutcommit_inode(inode, 1);
+ if (err < 0)
+ ret = err;
+ }
+#endif /* CONFIG_NFS_V4_1 */
+ return ret;
}
/*
@@ -1562,10 +1564,6 @@ int nfs_wb_all(struct inode *inode)
};
ret = sync_inode(inode, &wbc);
-#ifdef CONFIG_NFS_V4_1
- if (!ret && layoutcommit_needed(NFS_I(inode)))
- ret = pnfs_layoutcommit_inode(inode, 1);
-#endif
return ret;
}
--
1.6.6
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
