Roland Dreier wrote:
> I actually think that ep->com.cm_id is always valid when > connect_reply_upcall() is called. But I'd have to test it more. I'd > rather not add this change unless someone convinces me that there > actually is a path where the cm_id null...Then would it make sense to change the code if (ep->com.cm_id) { PDBG("%s ep %p tid %u status %d\n", __func__, ep, ep->hwtid, status); ep->com.cm_id->event_handler(ep->com.cm_id, &event); } to just PDBG("%s ep %p tid %u status %d\n", __func__, ep, ep->hwtid, status); ep->com.cm_id->event_handler(ep->com.cm_id, &event); ?
Yes. I haven't tested it though. But if you wanna push it, then I'll test it here soon.
Steve. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
