Would it be more appropriate that I rephrase it something along the lines ...
"We will simply drop the packet and let the remote end handle the dropped 
packet"

Ok - this makes more sense to me.

+    rx_ptr = priv->cm.rx_index_table[index];
+    if (unlikely(!rx_ptr)) {
+        ipoib_warn(priv, "Received packet from a connection "
+               "that is going away. Hardware will handle it.\n");
+        return;
+    }
If this check can ever succeed, then it's also possible for rx_ptr to
reference the wrong connection.  rx_table[index] should not be freed
until all receives associated with that QP have been processed.

rx_index_table[index] is freed only in the stale task. So, that means
all receives have been processed by this time.

Then it sounds like we can remove this check.

- Sean
_______________________________________________
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

Reply via email to