* [PATCH] scsi: target: remove redundant assignment to variable ret
@ 2021-07-21 10:15 Colin King
2021-07-27 3:09 ` Martin K. Petersen
2021-07-29 3:38 ` Martin K. Petersen
0 siblings, 2 replies; 3+ messages in thread
From: Colin King @ 2021-07-21 10:15 UTC (permalink / raw)
To: Martin K . Petersen, Varun Prakash, Mike Christie, linux-scsi,
target-devel
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
The variable ret is being initialized with a value that is never
read, the assignment is redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/target/iscsi/cxgbit/cxgbit_ddp.c | 2 +-
drivers/target/loopback/tcm_loop.c | 4 ++--
drivers/target/target_core_iblock.c | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/target/iscsi/cxgbit/cxgbit_ddp.c b/drivers/target/iscsi/cxgbit/cxgbit_ddp.c
index b044999ad002..072afd070f3e 100644
--- a/drivers/target/iscsi/cxgbit/cxgbit_ddp.c
+++ b/drivers/target/iscsi/cxgbit/cxgbit_ddp.c
@@ -234,7 +234,7 @@ cxgbit_get_r2t_ttt(struct iscsi_conn *conn, struct iscsi_cmd *cmd,
struct cxgbit_device *cdev = csk->com.cdev;
struct cxgbit_cmd *ccmd = iscsit_priv_cmd(cmd);
struct cxgbi_task_tag_info *ttinfo = &ccmd->ttinfo;
- int ret = -EINVAL;
+ int ret;
if ((!ccmd->setup_ddp) ||
(!test_bit(CSK_DDP_ENABLE, &csk->com.flags)))
diff --git a/drivers/target/loopback/tcm_loop.c b/drivers/target/loopback/tcm_loop.c
index 6d0b0e67e79e..fdc36274cb39 100644
--- a/drivers/target/loopback/tcm_loop.c
+++ b/drivers/target/loopback/tcm_loop.c
@@ -241,7 +241,7 @@ static int tcm_loop_abort_task(struct scsi_cmnd *sc)
{
struct tcm_loop_hba *tl_hba;
struct tcm_loop_tpg *tl_tpg;
- int ret = FAILED;
+ int ret;
/*
* Locate the tcm_loop_hba_t pointer
@@ -261,7 +261,7 @@ static int tcm_loop_device_reset(struct scsi_cmnd *sc)
{
struct tcm_loop_hba *tl_hba;
struct tcm_loop_tpg *tl_tpg;
- int ret = FAILED;
+ int ret;
/*
* Locate the tcm_loop_hba_t pointer
diff --git a/drivers/target/target_core_iblock.c b/drivers/target/target_core_iblock.c
index 44d9d028f716..4069a1edcfa3 100644
--- a/drivers/target/target_core_iblock.c
+++ b/drivers/target/target_core_iblock.c
@@ -83,7 +83,7 @@ static int iblock_configure_device(struct se_device *dev)
struct blk_integrity *bi;
fmode_t mode;
unsigned int max_write_zeroes_sectors;
- int ret = -ENOMEM;
+ int ret;
if (!(ib_dev->ibd_flags & IBDF_HAS_UDEV_PATH)) {
pr_err("Missing udev_path= parameters for IBLOCK\n");
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] scsi: target: remove redundant assignment to variable ret
2021-07-21 10:15 [PATCH] scsi: target: remove redundant assignment to variable ret Colin King
@ 2021-07-27 3:09 ` Martin K. Petersen
2021-07-29 3:38 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2021-07-27 3:09 UTC (permalink / raw)
To: Colin King
Cc: Martin K . Petersen, Varun Prakash, Mike Christie, linux-scsi,
target-devel, kernel-janitors, linux-kernel
Colin,
> The variable ret is being initialized with a value that is never read,
> the assignment is redundant and can be removed.
Applied to 5.15/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] scsi: target: remove redundant assignment to variable ret
2021-07-21 10:15 [PATCH] scsi: target: remove redundant assignment to variable ret Colin King
2021-07-27 3:09 ` Martin K. Petersen
@ 2021-07-29 3:38 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2021-07-29 3:38 UTC (permalink / raw)
To: linux-scsi, Colin King, Varun Prakash, target-devel, Mike Christie
Cc: Martin K . Petersen, linux-kernel, kernel-janitors
On Wed, 21 Jul 2021 11:15:19 +0100, Colin King wrote:
> The variable ret is being initialized with a value that is never
> read, the assignment is redundant and can be removed.
Applied to 5.15/scsi-queue, thanks!
[1/1] scsi: target: remove redundant assignment to variable ret
https://git.kernel.org/mkp/scsi/c/8f13142ac2eb
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-29 3:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-21 10:15 [PATCH] scsi: target: remove redundant assignment to variable ret Colin King
2021-07-27 3:09 ` Martin K. Petersen
2021-07-29 3:38 ` Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).