diff mbox series

[22/40] qlogicfas408: translate message to host byte status

Message ID	20210427083046.31620-23-hare@suse.de (mailing list archive)
State	Changes Requested
Headers	show Return-Path: <linux-scsi-owner@kernel.org> From: Hannes Reinecke <hare@suse.de> To: "Martin K. Petersen" <martin.petersen@oracle.com> Cc: Christoph Hellwig <hch@lst.de>, James Bottomley <james.bottomley@hansenpartnership.com>, Bart van Assche <bvanassche@acm.org>, linux-scsi@vger.kernel.org, Hannes Reinecke <hare@suse.de> Subject: [PATCH 22/40] qlogicfas408: translate message to host byte status Date: Tue, 27 Apr 2021 10:30:28 +0200 Message-Id: <20210427083046.31620-23-hare@suse.de> In-Reply-To: <20210427083046.31620-1-hare@suse.de> References: <20210427083046.31620-1-hare@suse.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	SCSI result cleanup, part 2 \| expand [PATCHv3,00/40] SCSI result cleanup, part 2 [01/40] st: return error code in st_scsi_execute() [02/40] scsi_ioctl: return error code when blk_rq_map_kern() fails [03/40] scsi: Fixup calling convention for scsi_mode_sense() [04/40] scsi: reshuffle response handling in scsi_mode_sense() [05/40] scsi_dh_alua: check for negative result value [06/40] scsi: stop using DRIVER_ERROR [07/40] scsi: introduce scsi_build_sense() [08/40] scsi: introduce scsi_status_is_check_condition() [09/40] scsi: Kill DRIVER_SENSE [10/40] scsi: do not use DRIVER_INVALID [11/40] scsi_error: use DID_TIME_OUT instead of DRIVER_TIMEOUT [12/40] xen-scsiback: use DID_ERROR instead of DRIVER_ERROR [13/40] xen-scsifront: compability status handling [14/40] scsi: Drop the now obsolete driver_byte definitions [15/40] NCR5380: Fold SCSI message ABORT onto DID_ABORT [16/40] scsi: add get_{status,host}_byte() accessor function [17/40] scsi: add scsi_msg_to_host_byte() [18/40] dc395: use standard macros to set SCSI result [19/40] dc395: translate message bytes [20/40] qlogicfas408: make ql_pcmd() a void function [21/40] qlogicfas408: whitespace cleanup [22/40] qlogicfas408: translate message to host byte status [23/40] nsp32: whitespace cleanup [24/40] nsp32: do not set message byte [25/40] wd33c93: translate message byte to host byte [26/40] mesh: translate message to host byte status [27/40] acornscsi: remove acornscsi_reportstatus() [28/40] acornscsi: translate message byte to host byte [29/40] aha152x: modify done() to use separate status bytes [30/40] aha152x: do not set message byte when calling scsi_done() [31/40] advansys: do not set message byte in SCSI status [32/40] fas216: translate message to host byte status [33/40] fas216: Use get_status_byte() to avoid using linux-specific status codes [34/40] FlashPoint: Use standard SCSI definitions [35/40] fdomain: drop last argument to fdomain_finish_cmd() [36/40] fdomain: translate message to host byte status [37/40] scsi: drop message byte helper [38/40] scsi: kill message byte [39/40] target: use standard SAM status types [40/40] scsi: drop obsolete linux-specific SCSI status codes

Commit Message

Hannes Reinecke April 27, 2021, 8:30 a.m. UTC

Instead of setting the message byte translate it to the appropriate
host byte. As error recovery would return DID_ERROR for any non-zero
message byte the translation doesn't change the error handling.

Signed-off-by: Hannes Reinecke <hare@suse.de>
---
 drivers/scsi/qlogicfas408.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/drivers/scsi/qlogicfas408.c b/drivers/scsi/qlogicfas408.c
index 86de400ca81a..34546cb99ccd 100644
--- a/drivers/scsi/qlogicfas408.c
+++ b/drivers/scsi/qlogicfas408.c
@@ -409,7 +409,8 @@  static void ql_pcmd(struct scsi_cmnd *cmd)
 	}
 
 	set_host_byte(cmd, result);
-	set_msg_byte(cmd, message);
+	if (result == DID_OK && message != COMMAND_COMPLETE)
+		scsi_msg_to_host_byte(cmd, message);
 	set_status_byte(cmd, status);
 	return;
 }