diff --git a/lib/iscsi/iscsi.c b/lib/iscsi/iscsi.c index c059e2021..7c6dc8ad6 100644 --- a/lib/iscsi/iscsi.c +++ b/lib/iscsi/iscsi.c @@ -2458,7 +2458,6 @@ iscsi_op_text(struct spdk_iscsi_conn *conn, struct spdk_iscsi_pdu *pdu) static int iscsi_op_logout(struct spdk_iscsi_conn *conn, struct spdk_iscsi_pdu *pdu) { - char buf[MAX_TMPBUF]; struct spdk_iscsi_pdu *rsp_pdu; uint32_t task_tag; uint32_t CmdSN; @@ -2553,38 +2552,36 @@ iscsi_op_logout(struct spdk_iscsi_conn *conn, struct spdk_iscsi_pdu *pdu) * login failed but initiator still sent a logout rather than * just closing the TCP connection. */ - snprintf(buf, sizeof buf, "Logout(login failed) from %s (%s) on" - " (%s:%s,%d)\n", - conn->initiator_name, conn->initiator_addr, - conn->portal_host, conn->portal_port, conn->pg_tag); + SPDK_DEBUGLOG(SPDK_LOG_ISCSI, "Logout(login failed) from %s (%s) on" + " (%s:%s,%d)\n", + conn->initiator_name, conn->initiator_addr, + conn->portal_host, conn->portal_port, conn->pg_tag); } else if (spdk_iscsi_param_eq_val(conn->sess->params, "SessionType", "Normal")) { - snprintf(buf, sizeof buf, "Logout from %s (%s) on %s tgt_node%d" - " (%s:%s,%d), ISID=%"PRIx64", TSIH=%u," - " CID=%u, HeaderDigest=%s, DataDigest=%s\n", - conn->initiator_name, conn->initiator_addr, - conn->target->name, conn->target->num, - conn->portal_host, conn->portal_port, conn->pg_tag, - conn->sess->isid, conn->sess->tsih, conn->cid, - (spdk_iscsi_param_eq_val(conn->params, "HeaderDigest", "CRC32C") - ? "on" : "off"), - (spdk_iscsi_param_eq_val(conn->params, "DataDigest", "CRC32C") - ? "on" : "off")); + SPDK_DEBUGLOG(SPDK_LOG_ISCSI, "Logout from %s (%s) on %s tgt_node%d" + " (%s:%s,%d), ISID=%"PRIx64", TSIH=%u," + " CID=%u, HeaderDigest=%s, DataDigest=%s\n", + conn->initiator_name, conn->initiator_addr, + conn->target->name, conn->target->num, + conn->portal_host, conn->portal_port, conn->pg_tag, + conn->sess->isid, conn->sess->tsih, conn->cid, + (spdk_iscsi_param_eq_val(conn->params, "HeaderDigest", "CRC32C") + ? "on" : "off"), + (spdk_iscsi_param_eq_val(conn->params, "DataDigest", "CRC32C") + ? "on" : "off")); } else { /* discovery session */ - snprintf(buf, sizeof buf, "Logout(discovery) from %s (%s) on" - " (%s:%s,%d), ISID=%"PRIx64", TSIH=%u," - " CID=%u, HeaderDigest=%s, DataDigest=%s\n", - conn->initiator_name, conn->initiator_addr, - conn->portal_host, conn->portal_port, conn->pg_tag, - conn->sess->isid, conn->sess->tsih, conn->cid, - (spdk_iscsi_param_eq_val(conn->params, "HeaderDigest", "CRC32C") - ? "on" : "off"), - (spdk_iscsi_param_eq_val(conn->params, "DataDigest", "CRC32C") - ? "on" : "off")); + SPDK_DEBUGLOG(SPDK_LOG_ISCSI, "Logout(discovery) from %s (%s) on" + " (%s:%s,%d), ISID=%"PRIx64", TSIH=%u," + " CID=%u, HeaderDigest=%s, DataDigest=%s\n", + conn->initiator_name, conn->initiator_addr, + conn->portal_host, conn->portal_port, conn->pg_tag, + conn->sess->isid, conn->sess->tsih, conn->cid, + (spdk_iscsi_param_eq_val(conn->params, "HeaderDigest", "CRC32C") + ? "on" : "off"), + (spdk_iscsi_param_eq_val(conn->params, "DataDigest", "CRC32C") + ? "on" : "off")); } - SPDK_NOTICELOG("%s", buf); - return 0; }