diff options
author | Matthew Wilcox <matthew@wil.cx> | 2007-09-09 10:56:32 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-12 14:47:48 -0400 |
commit | 7686f02aa8a380b3b0322179090f3c83ee4df977 (patch) | |
tree | d7c6e94add37eecef0bd9621500b28f52dbebaf8 | |
parent | b66229258f88a06ecdb8998066861b3ad8ad6a2a (diff) |
[SCSI] advansys: Remove a check for an impossible condition
The midlayer guarantees it won't call ->queuecommand for a host which
is handling a reset condition.
Signed-off-by: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r-- | drivers/scsi/advansys.c | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c index 526239a21332..cd17f28fd3ba 100644 --- a/drivers/scsi/advansys.c +++ b/drivers/scsi/advansys.c | |||
@@ -3281,25 +3281,6 @@ advansys_queuecommand(struct scsi_cmnd *scp, void (*done) (struct scsi_cmnd *)) | |||
3281 | /* against own ISR */ | 3281 | /* against own ISR */ |
3282 | spin_lock_irqsave(&boardp->lock, flags); | 3282 | spin_lock_irqsave(&boardp->lock, flags); |
3283 | 3283 | ||
3284 | /* | ||
3285 | * Block new commands while handling a reset or abort request. | ||
3286 | */ | ||
3287 | if (boardp->flags & ASC_HOST_IN_RESET) { | ||
3288 | ASC_DBG1(1, | ||
3289 | "advansys_queuecommand: scp 0x%lx blocked for reset request\n", | ||
3290 | (ulong)scp); | ||
3291 | scp->result = HOST_BYTE(DID_RESET); | ||
3292 | |||
3293 | /* | ||
3294 | * Add blocked requests to the board's 'done' queue. The queued | ||
3295 | * requests will be completed at the end of the abort or reset | ||
3296 | * handling. | ||
3297 | */ | ||
3298 | asc_enqueue(&boardp->done, scp, ASC_BACK); | ||
3299 | spin_unlock_irqrestore(&boardp->lock, flags); | ||
3300 | return 0; | ||
3301 | } | ||
3302 | |||
3303 | scp->scsi_done = done; | 3284 | scp->scsi_done = done; |
3304 | asc_res = asc_execute_scsi_cmnd(scp); | 3285 | asc_res = asc_execute_scsi_cmnd(scp); |
3305 | switch (asc_res) { | 3286 | switch (asc_res) { |