Overall, looks mostly fine (although as I've said a lot of times today,
please run scripts/checkpatch.pl and look at the output before sending
patches -- it saves lots of time for everyone).

One question -- given this:

 > +    if (skb->protocol != htons(ETH_P_IP))
 > +            return -1;

is there any way we could have anything other than IPv4 here:

 > +    if (iph->protocol != IPPROTO_TCP || iph->version != 4)
 > +            return -1;

 - 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

Reply via email to