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 kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
