| Message ID | 20230330061812.479463-1-markzhang@nvidia.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [rdma-next] RDMA/cm: Trace icm_send_rej event before the cm state is reset | expand |
On Thu, Mar 30, 2023 at 09:18:12AM +0300, Mark Zhang wrote: > Trace icm_send_rej event before the cm state is reset to idle, so that > correct cm state will be logged. For example when an incoming request is > rejected, the old trace log was: > icm_send_rej: local_id=961102742 remote_id=3829151631 state=IDLE reason=REJ_CONSUMER_DEFINED > With this patch: > icm_send_rej: local_id=312971016 remote_id=3778819983 state=MRA_REQ_SENT reason=REJ_CONSUMER_DEFINED > > Signed-off-by: Mark Zhang <markzhang@nvidia.com> > --- > drivers/infiniband/core/cm.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Please add the following Fixes line: Fixes: 8dc105befe16 ("RDMA/cm: Add tracepoints to track MAD send operations") and add the author of fixed patch to Cc list. Thanks > > diff --git a/drivers/infiniband/core/cm.c b/drivers/infiniband/core/cm.c > index 603c0aecc361..ff58058aeadc 100644 > --- a/drivers/infiniband/core/cm.c > +++ b/drivers/infiniband/core/cm.c > @@ -2912,6 +2912,8 @@ static int cm_send_rej_locked(struct cm_id_private *cm_id_priv, > (ari && ari_length > IB_CM_REJ_ARI_LENGTH)) > return -EINVAL; > > + trace_icm_send_rej(&cm_id_priv->id, reason); > + > switch (state) { > case IB_CM_REQ_SENT: > case IB_CM_MRA_REQ_RCVD: > @@ -2942,7 +2944,6 @@ static int cm_send_rej_locked(struct cm_id_private *cm_id_priv, > return -EINVAL; > } > > - trace_icm_send_rej(&cm_id_priv->id, reason); > ret = ib_post_send_mad(msg, NULL); > if (ret) { > cm_free_msg(msg); > -- > 2.37.1 >
diff --git a/drivers/infiniband/core/cm.c b/drivers/infiniband/core/cm.c index 603c0aecc361..ff58058aeadc 100644 --- a/drivers/infiniband/core/cm.c +++ b/drivers/infiniband/core/cm.c @@ -2912,6 +2912,8 @@ static int cm_send_rej_locked(struct cm_id_private *cm_id_priv, (ari && ari_length > IB_CM_REJ_ARI_LENGTH)) return -EINVAL; + trace_icm_send_rej(&cm_id_priv->id, reason); + switch (state) { case IB_CM_REQ_SENT: case IB_CM_MRA_REQ_RCVD: @@ -2942,7 +2944,6 @@ static int cm_send_rej_locked(struct cm_id_private *cm_id_priv, return -EINVAL; } - trace_icm_send_rej(&cm_id_priv->id, reason); ret = ib_post_send_mad(msg, NULL); if (ret) { cm_free_msg(msg);
Trace icm_send_rej event before the cm state is reset to idle, so that correct cm state will be logged. For example when an incoming request is rejected, the old trace log was: icm_send_rej: local_id=961102742 remote_id=3829151631 state=IDLE reason=REJ_CONSUMER_DEFINED With this patch: icm_send_rej: local_id=312971016 remote_id=3778819983 state=MRA_REQ_SENT reason=REJ_CONSUMER_DEFINED Signed-off-by: Mark Zhang <markzhang@nvidia.com> --- drivers/infiniband/core/cm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)