diff options
| author | Colin Ian King <colin.i.king@gmail.com> | 2024-04-10 17:22:21 +0100 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-04-11 10:21:24 +0200 |
| commit | fd0a133ef6edb8e49e74af505b38c93af4ba0b1e (patch) | |
| tree | 364ccb93de54d8a7276bb96d4ceda53ea8dba3c1 | |
| parent | 8bfb40be31ddea0cb4664b352e1797cfe6c91976 (diff) | |
| download | vfs-fd0a133ef6edb8e49e74af505b38c93af4ba0b1e.tar.gz | |
fs/direct-io: remove redundant assignment to variable retval
The variable retval is being assigned a value that is not being read,
it is being re-assigned later on in the function. The assignment
is redundant and can be removed.
Cleans up clang scan build warning:
fs/direct-io.c:1220:2: warning: Value stored to 'retval' is never
read [deadcode.DeadStores]
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20240410162221.292485-1-colin.i.king@gmail.com
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/direct-io.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/direct-io.c b/fs/direct-io.c index 62c97ff9e852a1..b0aafe640fa428 100644 --- a/fs/direct-io.c +++ b/fs/direct-io.c @@ -1217,7 +1217,6 @@ ssize_t __blockdev_direct_IO(struct kiocb *iocb, struct inode *inode, */ inode_dio_begin(inode); - retval = 0; sdio.blkbits = blkbits; sdio.blkfactor = i_blkbits - blkbits; sdio.block_in_file = offset >> blkbits; |