> > > Remove the complicated logic to free the cm_id resources in iw_cm event > > handlers vs when an application thread destroys the device. I'm not sure > > why this code was written, but simply allowing the last deref to free > > the memory is cleaner. It also prevents a deadlock when applications > > try to destroy cm_id's in their cm event handler function. > > The description here is misleading. we can never destroy the cm_id > inside the cm_id handler. Also, I don't think the deadlock was on cm_id > removal but rather on the qp referenced by the cm_id. I think the change > log can be improved. > I'll reword it. > The patch looks fine to me. -- 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