Fix log message in completion path.
authorJames Smart <jsmart2021@gmail.com>
Fri, 21 Apr 2017 23:04:48 +0000 (16:04 -0700)
committerChristoph Hellwig <hch@lst.de>
Mon, 24 Apr 2017 07:25:48 +0000 (09:25 +0200)
In the lpfc_nvme_io_cmd_wqe_cmpl routine the driver was printing two
pointers and the DID for the rport whenever an IO completed on a now
that had transitioned to a non active state.

There is no need to print the node pointer address for a node that
is not active the DID should be enough to debug.

Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: James Smart <james.smart@broadcom.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
drivers/scsi/lpfc/lpfc_nvme.c

index cbf3fe3f8637a25ee5d329b1a552544d98a769d8..ee1a285d0820977e83a01e79b8593511183ce1fe 100644 (file)
@@ -788,9 +788,8 @@ lpfc_nvme_io_cmd_wqe_cmpl(struct lpfc_hba *phba, struct lpfc_iocbq *pwqeIn,
        ndlp = rport->ndlp;
        if (!ndlp || !NLP_CHK_NODE_ACT(ndlp)) {
                lpfc_printf_vlog(vport, KERN_ERR, LOG_NODE | LOG_NVME_IOERR,
-                                "6061 rport %p, ndlp %p, DID x%06x ndlp "
-                                "not ready.\n",
-                                rport, ndlp, rport->remoteport->port_id);
+                                "6061 rport %p,  DID x%06x node not ready.\n",
+                                rport, rport->remoteport->port_id);
 
                ndlp = lpfc_findnode_did(vport, rport->remoteport->port_id);
                if (!ndlp) {