Michael S. Tsirkin wrote:
+void ipoib_cm_handle_rx_wc(struct net_device *dev, struct ib_wc *wc) +{ + struct ipoib_dev_priv *priv = netdev_priv(dev); + + if (priv->cm.srq) + handle_rx_wc_srq(dev, wc); + else + handle_rx_wc_nosrq(dev, wc); }I still think this conditional branch on datapath should be avoided by using separate RX handlers for SRQ/non SRQ cases. And same for the one on alloc_rx_skb.
I attempted implementing this. With NAPI now included, the code looked real ugly and so decided not do so.
Pradeep _______________________________________________ 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
