diff options
author | Chad Dupuis <chad.dupuis@qlogic.com> | 2011-08-16 14:29:26 -0400 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2011-08-27 10:19:56 -0400 |
commit | bc91ade9b7bc274d625c9b24c04d365a2daf481e (patch) | |
tree | d4f23db50e5afc7a3e669f09416f43ec8847778d | |
parent | 7594206493880007fd68a18d6e9f380a1afe20d4 (diff) |
[SCSI] qla2xxx: Double check for command completion if abort mailbox command fails.
Close a small window where we could falsely fail an abort request if the mailbox
command fails but the command was returned during interrupt context.
Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
-rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 2caab83c4c9f..4cace3f20c04 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
@@ -918,6 +918,10 @@ qla2xxx_eh_abort(struct scsi_cmnd *cmd) | |||
918 | qla2x00_sp_compl(ha, sp); | 918 | qla2x00_sp_compl(ha, sp); |
919 | spin_unlock_irqrestore(&ha->hardware_lock, flags); | 919 | spin_unlock_irqrestore(&ha->hardware_lock, flags); |
920 | 920 | ||
921 | /* Did the command return during mailbox execution? */ | ||
922 | if (ret == FAILED && !CMD_SP(cmd)) | ||
923 | ret = SUCCESS; | ||
924 | |||
921 | /* Wait for the command to be returned. */ | 925 | /* Wait for the command to be returned. */ |
922 | if (wait) { | 926 | if (wait) { |
923 | if (qla2x00_eh_wait_on_command(cmd) != QLA_SUCCESS) { | 927 | if (qla2x00_eh_wait_on_command(cmd) != QLA_SUCCESS) { |