scsi: core: return BLK_STS_OK for DID_OK in __scsi_error_from_host_byte()

hreinecke · martinkpetersen · commit e39a97353e53 · 2018-03-01T20:16:52.000-05:00
When converting __scsi_error_from_host_byte() to BLK_STS error codes the case DID_OK was forgotten, resulting in it always returning an error. Fixes: 2a842ac ("block: introduce new block status code type") Cc: Doug Gilbert <dgilbert@interlog.com> Signed-off-by: Hannes Reinecke <hare@suse.com> Reviewed-by: Douglas Gilbert <dgilbert@interlog.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
@@ -720,6 +720,8 @@ static blk_status_t __scsi_error_from_host_byte(struct scsi_cmnd *cmd,
 		int result)
 {
 	switch (host_byte(result)) {
+	case DID_OK:
+		return BLK_STS_OK;
 	case DID_TRANSPORT_FAILFAST:
 		return BLK_STS_TRANSPORT;
 	case DID_TARGET_FAILURE:

Original file line number	Diff line number	Diff line change
`@@ -720,6 +720,8 @@ static blk_status_t __scsi_error_from_host_byte(struct scsi_cmnd *cmd,`
`720`	`720`	`int result)`
`721`	`721`	`{`
`722`	`722`	`switch (host_byte(result)) {`
	`723`	`+ case DID_OK:`
	`724`	`+ return BLK_STS_OK;`
`723`	`725`	`case DID_TRANSPORT_FAILFAST:`
`724`	`726`	`return BLK_STS_TRANSPORT;`
`725`	`727`	`case DID_TARGET_FAILURE:`