On Thu, Feb 17, 2011 at 10:44 AM, Ben Pfaff <[email protected]> wrote:
> It seems to me that vlan_deaccel_tag() needs to increase csum_start by
> 4 in the CHECKSUM_PARTIAL case, something like this following.  I
> don't see anything like this in the other callers of __vlan_put_tag(),
> but it looks like most of those wouldn't have CHECKSUM_PARTIAL skbs
> anyway.
>
> Does this seem correct to you?

This shouldn't be necessary because csum_start is relative to
skb->head not skb->data, so you can push and pop headers on the front
of the packet without having to update it.

Are you seeing problems?

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev_openvswitch.org

Reply via email to