[ovs-build] Still Failing: istokes/ovs#1125 (master - 56c3de3)
builds at travis-ci.com
Mon Nov 1 14:58:23 UTC 2021
Build Update for istokes/ovs
Status: Still Failing
Duration: 16 mins and 2 secs
Commit: 56c3de3 (master)
Author: Wilson Peng
Message: datapath-windows:Reset PseudoChecksum value only for TX direction offload case
While testing OVS-windows flows for the DNAT/SNAT action, the checksum in
TCP header is set incorrectly when TCP offload is enabled by default. As a
result,the packet will be dropped on the Windows VM when processing the packet
from Linux VM which has included correct checksum at first. On the Windows VM,
it has gone through two NAT actions and OVS Windows kernel will reset the
checksum to PseudoChecksum and then it will lose the original correct checksum
value which is set outside.
Back to the Nat TCP/UDP checksum value reset logic,it should reset it TCP checksum
To be PseudoChecksum value only on Tx direction for TCP Offload case. For the packet
>From the outside, OVS Windows Kernel does not need reset the TCP/UDP checksum as
It should be the job of the received network driver to get out a correct checksum
>>>sample flow on default configuration on both Windows VM and Linux VM
Dst=10.176.26.107) Without the fix the return back packet(src=10.176.26.107,
Dst=169.254.169.253) will have the correct TCP checksum. After the reverse NAT
Actions, it will be changed to be packet (src=10.110.225.146, Dst=192.168.252.1)
But with incorrect TCP checksum 0xa97a which is
The PseudoChecksum. Related packet is put on the reported issue below.
Signed-off-by: Wilson Peng <pweisong at vmware.com>
Signed-off-by: Alin-Gabriel Serdean <aserdean at ovn.org>
View the changeset: https://github.com/istokes/ovs/compare/ae2424696cff...56c3de3c61e6
View the full build log and details: https://app.travis-ci.com/github/istokes/ovs/builds/240950698?utm_medium=notification&utm_source=email
You can unsubscribe from build emails from the istokes/ovs repository going to https://app.travis-ci.com/account/preferences/unsubscribe?repository=16902882&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://app.travis-ci.com/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the build