diff options
| author |   Lukas Bulwahn <lukas.bulwahn@gmail.com> | 2022-08-10 13:40:01 +0200 |
|---|---|---|
| committer |   Anna Schumaker <Anna.Schumaker@Netapp.com> | 2022-10-03 11:26:36 -0400 |
| commit | 384edeb46f07f4ee1b3adda9416e724421e2fad5 (patch) | |
| tree | 61c1c5d30176e496d6487aef81df02f79fbdd95d /fs/nfs | |
| parent | nfs: remove unnecessary (void*) conversions. (diff) | |
| download | linux-dev-384edeb46f07f4ee1b3adda9416e724421e2fad5.tar.xz linux-dev-384edeb46f07f4ee1b3adda9416e724421e2fad5.zip | |
NFS: clean up a needless assignment in nfs_file_write()
Commit 064109db53ec ("NFS: remove redundant code in nfs_file_write()")
identifies that filemap_fdatawait_range() will always return 0 and removes
a dead error-handling case in nfs_file_write(). With this change however,
assigning the return of filemap_fdatawait_range() to the result variable is
a dead store.
Remove this needless assignment.
No functional change. No change in object code.
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'fs/nfs')
| -rw-r--r-- | fs/nfs/file.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/nfs/file.c b/fs/nfs/file.c index e032fe201a36..7a4d2fe9c0b3 100644 --- a/fs/nfs/file.c +++ b/fs/nfs/file.c @@ -655,9 +655,9 @@ ssize_t nfs_file_write(struct kiocb *iocb, struct iov_iter *from) goto out; } if (mntflags & NFS_MOUNT_WRITE_WAIT) { - result = filemap_fdatawait_range(file->f_mapping, - iocb->ki_pos - written, - iocb->ki_pos - 1); + filemap_fdatawait_range(file->f_mapping, + iocb->ki_pos - written, + iocb->ki_pos - 1); } result = generic_write_sync(iocb, written); if (result < 0) |