On Tue, Jul 10, 2018 at 09:42:10AM +0000, Bason, Yuval wrote: > > > @@ -2279,8 +2276,9 @@ int qedr_destroy_qp(struct ib_qp *ibqp) > > > > > > qedr_free_qp_resources(dev, qp); > > > > > > - if (atomic_dec_and_test(&qp->refcnt)) { > > > - qedr_idr_remove(dev, qp->qp_id); > > > + if (atomic_dec_and_test(&qp->refcnt) && > > > + rdma_protocol_iwarp(&dev->ibdev, 1)) { > > > > Please send a cleanup patch to use refcount_t for this stuff? > Do you mean as part of this patch series? No, just in general.. Jason -- 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