diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-09-11 15:38:32 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-09-11 15:38:32 -0400 |
| commit | 7b281ee026552f10862b617a2a51acf49c829554 (patch) | |
| tree | c86ce80fa069cbe60f3c3dfbc939269ecc0d1aa4 /fs/nfs/file.c | |
| parent | f39c1bfb5a03e2d255451bff05be0d7255298fa4 (diff) | |
| download | blackbird-op-linux-7b281ee026552f10862b617a2a51acf49c829554.tar.gz blackbird-op-linux-7b281ee026552f10862b617a2a51acf49c829554.zip | |
NFS: fsync() must exit with an error if page writeback failed
We need to ensure that if the call to filemap_write_and_wait_range()
fails, then we report that error back to the application.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/file.c')
| -rw-r--r-- | fs/nfs/file.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/nfs/file.c b/fs/nfs/file.c index 75d6d0a3d32e..6a7fcab7ecb3 100644 --- a/fs/nfs/file.c +++ b/fs/nfs/file.c @@ -287,10 +287,12 @@ nfs_file_fsync(struct file *file, loff_t start, loff_t end, int datasync) struct inode *inode = file->f_path.dentry->d_inode; ret = filemap_write_and_wait_range(inode->i_mapping, start, end); + if (ret != 0) + goto out; mutex_lock(&inode->i_mutex); ret = nfs_file_fsync_commit(file, start, end, datasync); mutex_unlock(&inode->i_mutex); - +out: return ret; } |
