> +        ip->ip_csum = csum(ip, IP_IHL(ip->ip_ihl_ver) * 4);

I think what you have here is correct, but would be a bit
safer/cleaner if we changed it to:

ip->ip_csum = csum(ip, sizeof *ip);

Ethan


>      } else if (flow->dl_type == htons(ETH_TYPE_IPV6)) {
>          /* XXX */
>      } else if (flow->dl_type == htons(ETH_TYPE_ARP)) {
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to