The ! has higher precedence than the & operation.  I've added
parenthesis so this works as intended.

Fixes: 952c303b329c ("scsi: lpfc: Ensure io aborts interlocked with the 
target.")
Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/drivers/scsi/lpfc/lpfc_nvme.c b/drivers/scsi/lpfc/lpfc_nvme.c
index 60f0dbd0f192..517ae570e507 100644
--- a/drivers/scsi/lpfc/lpfc_nvme.c
+++ b/drivers/scsi/lpfc/lpfc_nvme.c
@@ -948,7 +948,7 @@ lpfc_nvme_io_cmd_wqe_cmpl(struct lpfc_hba *phba, struct 
lpfc_iocbq *pwqeIn,
        /* NVME targets need completion held off until the abort exchange
         * completes.
         */
-       if (!lpfc_ncmd->flags & LPFC_SBUF_XBUSY)
+       if (!(lpfc_ncmd->flags & LPFC_SBUF_XBUSY))
                nCmd->done(nCmd);
 
        spin_lock_irqsave(&phba->hbalock, flags);

Reply via email to