On Tue, Jun 27, 2017 at 12:19:38PM +0300, Sagi Grimberg wrote: > From: Vijay Immanuel <vijayi@xxxxxxxxxxxxxxxxx> > > If we modified the qp to ERROR state, and > drained the recieve queue, post_recv must > trigger the responder task to complete > the drain work request. > > Cc: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Signed-off-by: Vijay Immanuel <vijayi@xxxxxxxxxxxxxxxxx> > Signed-off-by: Sagi Grimberg <sagi@xxxxxxxxxxx> > --- > drivers/infiniband/sw/rxe/rxe_resp.c | 3 +++ > drivers/infiniband/sw/rxe/rxe_verbs.c | 3 +++ > 2 files changed, 6 insertions(+) > It looks OK for me. Moni ? > diff --git a/drivers/infiniband/sw/rxe/rxe_resp.c b/drivers/infiniband/sw/rxe/rxe_resp.c > index 23039768f541..f66b48c114ac 100644 > --- a/drivers/infiniband/sw/rxe/rxe_resp.c > +++ b/drivers/infiniband/sw/rxe/rxe_resp.c > @@ -1217,6 +1217,9 @@ void rxe_drain_req_pkts(struct rxe_qp *qp, bool notify) > kfree_skb(skb); > } > > + if (notify) > + return; > + > while (!qp->srq && qp->rq.queue && queue_head(qp->rq.queue)) > advance_consumer(qp->rq.queue); > } > diff --git a/drivers/infiniband/sw/rxe/rxe_verbs.c b/drivers/infiniband/sw/rxe/rxe_verbs.c > index 83d709e74dfb..94af4d67988b 100644 > --- a/drivers/infiniband/sw/rxe/rxe_verbs.c > +++ b/drivers/infiniband/sw/rxe/rxe_verbs.c > @@ -919,6 +919,9 @@ static int rxe_post_recv(struct ib_qp *ibqp, struct ib_recv_wr *wr, > > spin_unlock_irqrestore(&rq->producer_lock, flags); > > + if (qp->resp.state == QP_STATE_ERROR) > + rxe_run_task(&qp->resp.task, 1); > + > err1: > return err; > } > -- > 2.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html
Attachment:
signature.asc
Description: PGP signature
