diff options
author | <jejb@titanic.il.steeleye.com> | 2005-04-12 18:49:09 -0400 |
---|---|---|
committer | James Bottomley <jejb@titanic> | 2005-04-18 13:35:06 -0400 |
commit | bf341919dbc1fbcbb565fb3224c840760ebd9f85 (patch) | |
tree | 9ff0bb375ccd709b018eccd6d338bf18b9c898aa | |
parent | 6f71d9bc025b02a8cbc2be83b0226a7043a507a5 (diff) |
scsi: add DID_REQUEUE to the error handling
We have a DID_IMM_RETRY to require a retry at once, but we could do with
a DID_REQUEUE to instruct the mid-layer to treat this command in the
same manner as QUEUE_FULL or BUSY (i.e. halt the submission until
another command returns ... or the queue pressure builds if there are no
outstanding commands).
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r-- | drivers/scsi/scsi_error.c | 3 | ||||
-rw-r--r-- | include/scsi/scsi.h | 2 |
2 files changed, 5 insertions, 0 deletions
diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index 13bac93fc397..203a0812508a 100644 --- a/drivers/scsi/scsi_error.c +++ b/drivers/scsi/scsi_error.c | |||
@@ -1306,6 +1306,9 @@ int scsi_decide_disposition(struct scsi_cmnd *scmd) | |||
1306 | case DID_IMM_RETRY: | 1306 | case DID_IMM_RETRY: |
1307 | return NEEDS_RETRY; | 1307 | return NEEDS_RETRY; |
1308 | 1308 | ||
1309 | case DID_REQUEUE: | ||
1310 | return ADD_TO_MLQUEUE; | ||
1311 | |||
1309 | case DID_ERROR: | 1312 | case DID_ERROR: |
1310 | if (msg_byte(scmd->result) == COMMAND_COMPLETE && | 1313 | if (msg_byte(scmd->result) == COMMAND_COMPLETE && |
1311 | status_byte(scmd->result) == RESERVATION_CONFLICT) | 1314 | status_byte(scmd->result) == RESERVATION_CONFLICT) |
diff --git a/include/scsi/scsi.h b/include/scsi/scsi.h index 3507b3d7c0eb..659ecf48fb4a 100644 --- a/include/scsi/scsi.h +++ b/include/scsi/scsi.h | |||
@@ -295,6 +295,8 @@ struct scsi_lun { | |||
295 | #define DID_PASSTHROUGH 0x0a /* Force command past mid-layer */ | 295 | #define DID_PASSTHROUGH 0x0a /* Force command past mid-layer */ |
296 | #define DID_SOFT_ERROR 0x0b /* The low level driver just wish a retry */ | 296 | #define DID_SOFT_ERROR 0x0b /* The low level driver just wish a retry */ |
297 | #define DID_IMM_RETRY 0x0c /* Retry without decrementing retry count */ | 297 | #define DID_IMM_RETRY 0x0c /* Retry without decrementing retry count */ |
298 | #define DID_REQUEUE 0x0d /* Requeue command (no immediate retry) also | ||
299 | * without decrementing the retry count */ | ||
298 | #define DRIVER_OK 0x00 /* Driver status */ | 300 | #define DRIVER_OK 0x00 /* Driver status */ |
299 | 301 | ||
300 | /* | 302 | /* |