diff options
author | James Bottomley | 2016-11-29 19:57:03 +0100 |
---|---|---|
committer | James Bottomley | 2016-11-29 19:57:03 +0100 |
commit | ff682a3dad98074f956e6da099448d3eb93f095c (patch) | |
tree | 493951eee8e72ff208c3e4e93ecf970a5458a8b4 | |
parent | Merge remote-tracking branch 'mkp-scsi/4.9/scsi-fixes' into fixes (diff) | |
parent | scsi: lpfc: fix oops/BUG in lpfc_sli_ringtxcmpl_put() (diff) | |
download | kernel-qcow2-linux-ff682a3dad98074f956e6da099448d3eb93f095c.tar.gz kernel-qcow2-linux-ff682a3dad98074f956e6da099448d3eb93f095c.tar.xz kernel-qcow2-linux-ff682a3dad98074f956e6da099448d3eb93f095c.zip |
Merge remote-tracking branch 'mkp-scsi/4.9/scsi-fixes' into fixes
-rw-r--r-- | drivers/scsi/lpfc/lpfc_sli.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c index c5326055beee..f4f77c5b0c83 100644 --- a/drivers/scsi/lpfc/lpfc_sli.c +++ b/drivers/scsi/lpfc/lpfc_sli.c @@ -1323,18 +1323,20 @@ lpfc_sli_ringtxcmpl_put(struct lpfc_hba *phba, struct lpfc_sli_ring *pring, { lockdep_assert_held(&phba->hbalock); - BUG_ON(!piocb || !piocb->vport); + BUG_ON(!piocb); list_add_tail(&piocb->list, &pring->txcmplq); piocb->iocb_flag |= LPFC_IO_ON_TXCMPLQ; if ((unlikely(pring->ringno == LPFC_ELS_RING)) && (piocb->iocb.ulpCommand != CMD_ABORT_XRI_CN) && - (piocb->iocb.ulpCommand != CMD_CLOSE_XRI_CN) && - (!(piocb->vport->load_flag & FC_UNLOADING))) - mod_timer(&piocb->vport->els_tmofunc, - jiffies + - msecs_to_jiffies(1000 * (phba->fc_ratov << 1))); + (piocb->iocb.ulpCommand != CMD_CLOSE_XRI_CN)) { + BUG_ON(!piocb->vport); + if (!(piocb->vport->load_flag & FC_UNLOADING)) + mod_timer(&piocb->vport->els_tmofunc, + jiffies + + msecs_to_jiffies(1000 * (phba->fc_ratov << 1))); + } return 0; } |