[SCSI] Set hostbyte status in scsi_check_sense()
authorHannes Reinecke <hare@suse.de>
Mon, 1 Jul 2013 13:16:24 +0000 (15:16 +0200)
committerJames Bottomley <JBottomley@Parallels.com>
Fri, 23 Aug 2013 16:34:56 +0000 (12:34 -0400)
We should be modifying the host_byte status in scsi_check_sense()
directly; this saves us to introduce a special return code for
each and every condition.

Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/scsi_error.c
include/scsi/scsi.h

index 21505962f5393d1cae00ca7f92f70cc131523f0c..7be5229ed3a4025fa359c334d7a8888e54b48708 100644 (file)
@@ -228,7 +228,7 @@ static inline void scsi_eh_prt_fail_stats(struct Scsi_Host *shost,
  * @scmd:      Cmd to have sense checked.
  *
  * Return value:
- *     SUCCESS or FAILED or NEEDS_RETRY or TARGET_ERROR
+ *     SUCCESS or FAILED or NEEDS_RETRY or ADD_TO_MLQUEUE
  *
  * Notes:
  *     When a deferred error is detected the current command has
@@ -359,13 +359,15 @@ static int scsi_check_sense(struct scsi_cmnd *scmd)
        case MISCOMPARE:
        case BLANK_CHECK:
        case DATA_PROTECT:
-               return TARGET_ERROR;
+               set_host_byte(scmd, DID_TARGET_FAILURE);
+               return SUCCESS;
 
        case MEDIUM_ERROR:
                if (sshdr.asc == 0x11 || /* UNRECOVERED READ ERR */
                    sshdr.asc == 0x13 || /* AMNF DATA FIELD */
                    sshdr.asc == 0x14) { /* RECORD NOT FOUND */
-                       return TARGET_ERROR;
+                       set_host_byte(scmd, DID_TARGET_FAILURE);
+                       return SUCCESS;
                }
                return NEEDS_RETRY;
 
@@ -373,14 +375,14 @@ static int scsi_check_sense(struct scsi_cmnd *scmd)
                if (scmd->device->retry_hwerror)
                        return ADD_TO_MLQUEUE;
                else
-                       return TARGET_ERROR;
+                       set_host_byte(scmd, DID_TARGET_FAILURE);
 
        case ILLEGAL_REQUEST:
                if (sshdr.asc == 0x20 || /* Invalid command operation code */
                    sshdr.asc == 0x21 || /* Logical block address out of range */
                    sshdr.asc == 0x24 || /* Invalid field in cdb */
                    sshdr.asc == 0x26) { /* Parameter value invalid */
-                       return TARGET_ERROR;
+                       set_host_byte(scmd, DID_TARGET_FAILURE);
                }
                return SUCCESS;
 
@@ -843,7 +845,6 @@ retry:
                case SUCCESS:
                case NEEDS_RETRY:
                case FAILED:
-               case TARGET_ERROR:
                        break;
                case ADD_TO_MLQUEUE:
                        rtn = NEEDS_RETRY;
@@ -1577,14 +1578,6 @@ int scsi_decide_disposition(struct scsi_cmnd *scmd)
                rtn = scsi_check_sense(scmd);
                if (rtn == NEEDS_RETRY)
                        goto maybe_retry;
-               else if (rtn == TARGET_ERROR) {
-                       /*
-                        * Need to modify host byte to signal a
-                        * permanent target failure
-                        */
-                       set_host_byte(scmd, DID_TARGET_FAILURE);
-                       rtn = SUCCESS;
-               }
                /* if rtn == FAILED, we have no sense information;
                 * returning FAILED will wake the error handler thread
                 * to collect the sense and redo the decide
index 4b87d99e7fa13645cc0df102447547f36a330eaa..e4cd3e97553884180fb934f1230f56886c312f56 100644 (file)
@@ -486,7 +486,6 @@ static inline int scsi_is_wlun(unsigned int lun)
 #define TIMEOUT_ERROR   0x2007
 #define SCSI_RETURN_NOT_HANDLED   0x2008
 #define FAST_IO_FAIL   0x2009
-#define TARGET_ERROR    0x200A
 
 /*
  * Midlevel queue return values.