Or Gerlitz wrote:
Vladimir Sokolovsky wrote:
add "ipoib_use_lro" module parameter to enable LRO.
How about that for this one we'll go on a finer granularity of being able to set it per device through sysfs?

Yes, we thought about this  and I am planning to add this granularity
later, in a different patch.


+    priv->lro.lro_mgr.get_skb_header = get_skb_hdr;
+    priv->lro.lro_mgr.ip_summed = CHECKSUM_UNNECESSARY;
+    priv->lro.lro_mgr.ip_summed_aggr = CHECKSUM_UNNECESSARY;
Looking in _lro_proc_skb() which is the code that would be processing ipoib skbs, it seems that the values of lro_mgr.ip_summed and lro_mgr.ip_summed_aggr should correlate to whether the HCA is capable of doing checksum offload or not.

Or.



+static int get_skb_hdr(struct sk_buff *skb, void **iphdr,
+               void **tcph, u64 *hdr_flags, void *priv)
+{

...

+
+    if (unlikely(skb->ip_summed != CHECKSUM_UNNECESSARY))
+        return -1;

This will return -1 if HCA is not capable of doing checksum offload.

Regards,
Vladimir


_______________________________________________
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