aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChad Dupuis <chad.dupuis@qlogic.com>2011-08-16 14:29:26 -0400
committerJames Bottomley <JBottomley@Parallels.com>2011-08-27 10:19:56 -0400
commitbc91ade9b7bc274d625c9b24c04d365a2daf481e (patch)
treed4f23db50e5afc7a3e669f09416f43ec8847778d
parent7594206493880007fd68a18d6e9f380a1afe20d4 (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.c4
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) {