> It is common practice to put a pointer/index to a per-QP > structure inside the wrid. This data is available after poll_cq > returns, when cq lock is not taken. If this pointer is used > directly inside the event handler, the ULP that is moving QP to > reset has no way to know when is it safe to free data it points to, > unless the verbs provider synchronizes with the IRQ handler > before the verbs returns.
Does this fix any problem for in-tree (or OFED) drivers? Because I'm not convinced this is something that a low-level driver should try to handle. A ULP that suffers from this that polls a CQ from a workqueue, say, rather than an interrupt remains broken even after this change. And my gut feeling is that this type of problem is something a ULP should handle by not getting into this situation in the first place. BTW, have you had a chance to test the other changes and see if they fix the IPoIB CM issue? - R. _______________________________________________ general mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
