Re: [Patch net-next] openvswitch: adjust skb_gso_segment() for rx path
From: Cong Wang <hidden>
Date: 2013-02-01 02:09:19
On Thu, 2013-01-31 at 17:09 -0800, Jesse Gross wrote:
I would argue that the meanings are the same on transmit and receive but the values that you are likely to see differ depending on the path. With bridging (which includes both Open vSwitch and the bridge) it's very difficult to know whether you are on the transmit or receive path. Personally, I would just remove the warning completely.
I would keep it as it is, I fixed several RHEL kernel bugs exposed by this warning, it is pretty useful for debugging. ...
You can just drop the change in the block above and gso_send_check() will take care of the rest. Regardless of the state of the incoming packet, GSO needs the checksum to be reset to CHECKSUM_PARTIAL in order to work.
Right, makes sense.
I think this is a different problem, unrelated to the checksum issue. Can you test this patch: http://git.kernel.org/?p=linux/kernel/git/jesse/openvswitch.git;a=commit;h=d9d59089c43fc33eb403cbb928e56c131f191dd5 I believe it should fix the problem, although there are still a few lingering reports that I'm trying to track down.
I will try your patch together with my updated patch. Thanks!