Skip to content

Commit daefad7

Browse files
Vu PhamMatt Benjamin
Vu Pham
authored and
Matt Benjamin
committed
xio: enable accelio debug on level 2
Enable accelio debug (mostly on connection) on level 2 and sync with XioConnection debug events Signed-off-by: Vu Pham <[email protected]> Signed-off-by: Matt Benjamin <[email protected]>
1 parent aa5f195 commit daefad7

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

src/msg/xio/XioConnection.cc

+6-2
Original file line numberDiff line numberDiff line change
@@ -456,6 +456,8 @@ int XioConnection::on_ow_msg_send_complete(struct xio_session *session,
456456
if ((send_ctr <= uint32_t(xio_qdepth_low_mark())) &&
457457
(1 /* XXX memory <= memory low-water mark */)) {
458458
cstate.state_up_ready(XioConnection::CState::OP_FLAG_NONE);
459+
ldout(msgr->cct,2) << "on_msg_delivered xcon: " << xmsg->xcon <<
460+
" session: " << session << " up_ready from flow_controlled" << dendl;
459461
}
460462
}
461463

@@ -466,15 +468,17 @@ int XioConnection::on_ow_msg_send_complete(struct xio_session *session,
466468

467469
void XioConnection::msg_send_fail(XioMsg *xmsg, int code)
468470
{
469-
ldout(msgr->cct,4) << "xio_send_msg FAILED " << &xmsg->req_0.msg << " code=" << code <<
471+
ldout(msgr->cct,2) << "xio_send_msg FAILED xcon: " << this <<
472+
" xmsg: " << &xmsg->req_0.msg << " code=" << code <<
470473
" (" << xio_strerror(code) << ")" << dendl;
471474
/* return refs taken for each xio_msg */
472475
xmsg->put_msg_refs();
473476
} /* msg_send_fail */
474477

475478
void XioConnection::msg_release_fail(struct xio_msg *msg, int code)
476479
{
477-
ldout(msgr->cct,4) << "xio_release_msg FAILED " << msg << "code=" << code <<
480+
ldout(msgr->cct,2) << "xio_release_msg FAILED xcon: " << this <<
481+
" xmsg: " << msg << "code=" << code <<
478482
" (" << xio_strerror(code) << ")" << dendl;
479483
} /* msg_release_fail */
480484

src/msg/xio/XioMessenger.cc

+7-4
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ static const level_pair LEVELS[] = {
5858
make_pair("fatal", 0),
5959
make_pair("error", 0),
6060
make_pair("warn", 1),
61-
make_pair("info", 5),
62-
make_pair("debug", 10),
61+
make_pair("info", 1),
62+
make_pair("debug", 2),
6363
make_pair("trace", 20)
6464
};
6565

@@ -399,7 +399,7 @@ int XioMessenger::session_event(struct xio_session *session,
399399
case XIO_SESSION_CONNECTION_ESTABLISHED_EVENT:
400400
xcon = static_cast<XioConnection*>(event_data->conn_user_context);
401401

402-
ldout(cct,4) << "connection established " << event_data->conn
402+
ldout(cct,2) << "connection established " << event_data->conn
403403
<< " session " << session << " xcon " << xcon << dendl;
404404

405405
/* notify hook */
@@ -450,7 +450,7 @@ int XioMessenger::session_event(struct xio_session *session,
450450
}
451451
break;
452452
case XIO_SESSION_CONNECTION_ERROR_EVENT:
453-
ldout(cct,4) << xio_session_event_types[event_data->event]
453+
ldout(cct,2) << xio_session_event_types[event_data->event]
454454
<< " user_context " << event_data->conn_user_context << dendl;
455455
/* informational (Eyal)*/
456456
break;
@@ -958,6 +958,9 @@ ConnectionRef XioMessenger::get_connection(const entity_inst_t& dest)
958958
/* XXXX pre-merge of session startup negotiation ONLY! */
959959
xcon->cstate.state_up_ready(XioConnection::CState::OP_FLAG_NONE);
960960

961+
ldout(cct,2) << "new connection xcon: " << xcon <<
962+
" up_ready on session " << xcon->session << dendl;
963+
961964
return xcon->get(); /* nref +1 */
962965
}
963966
} /* get_connection */

0 commit comments

Comments
 (0)