scsi: ibmvscsi_tgt: remove set but not used variables 'iue' and 'sd' (4aca8fe7) · Commits · 戴 / test

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c: In function ibmvscsis_send_messages: drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c:1888:19: warning: variable iue set but not used [-Wunused-but-set-variable] drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c: In function ibmvscsis_queue_data_in: drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c:3806:8: warning: variable sd set but not used [-Wunused-but-set-variable] Link: https://lore.kernel.org/r/20191213064042.161840-1-chenzhou10@huawei.com Reported-by:

Hulk Robot <hulkci@huawei.com> Signed-off-by:

Chen Zhou <chenzhou10@huawei.com> Signed-off-by:

Martin K. Petersen <martin.petersen@oracle.com>

drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c

+0 −5

Original line number	Diff line number	Diff line
		@@ -1877,7 +1877,6 @@ static void ibmvscsis_send_messages(struct scsi_info *vscsi)
		*/
		struct viosrp_crq crq = (struct viosrp_crq )&msg_hi;
		struct ibmvscsis_cmd cmd, nxt;
		struct iu_entry *iue;
		long rc = ADAPT_SUCCESS;
		bool retry = false;

		@@ -1931,8 +1930,6 @@ static void ibmvscsis_send_messages(struct scsi_info *vscsi)
		*/
		vscsi->credit += 1;
		} else {
		iue = cmd->iue;

		crq->valid = VALID_CMD_RESP_EL;
		crq->format = cmd->rsp.format;

		@@ -3796,7 +3793,6 @@ static int ibmvscsis_queue_data_in(struct se_cmd *se_cmd)
		se_cmd);
		struct iu_entry *iue = cmd->iue;
		struct scsi_info *vscsi = cmd->adapter;
		char *sd;
		uint len = 0;
		int rc;

		@@ -3804,7 +3800,6 @@ static int ibmvscsis_queue_data_in(struct se_cmd *se_cmd)
		1);
		if (rc) {
		dev_err(&vscsi->dev, "srp_transfer_data failed: %d\n", rc);
		sd = se_cmd->sense_buffer;
		se_cmd->scsi_sense_length = 18;
		memset(se_cmd->sense_buffer, 0, se_cmd->scsi_sense_length);
		/* Logical Unit Communication Time-out asc/ascq = 0x0801 */

Admin message