[ovs-build] Passed: gvrose8192/ovs-experimental#25 (force-commit-fix - 07d55c8)

Travis CI builds at travis-ci.org
Thu Jul 13 16:13:43 UTC 2017

Build Update for gvrose8192/ovs-experimental

Build: #25
Status: Passed

Duration: 1 hour, 22 minutes, and 46 seconds
Commit: 07d55c8 (force-commit-fix)
Author: Greg Rose
Message: datapath: Fix for force/commit action failures

When there is an established connection in direction A->B, it is
possible to receive a packet on port B which then executes
ct(commit,force) without first performing ct() - ie, a lookup.
In this case, we would expect that this packet can delete the existing
entry so that we can commit a connection with direction B->A. However,
currently we only perform a check in skb_nfct_cached() for whether
OVS_CS_F_TRACKED is set and OVS_CS_F_INVALID is not set, ie that a
lookup previously occurred. In the above scenario, a lookup has not
occurred but we should still be able to statelessly look up the
existing entry and potentially delete the entry if it is in the
opposite direction.

This patch extends the check to also hint that if the action has the
force flag set, then we will lookup the existing entry so that the
force check at the end of skb_nfct_cached has the ability to delete
the connection.

Co-authored-by: Joe Stringer <joe at ovn.org>
Signed-off-by: Joe Stringer <joe at ovn.org>
Signed-off-by: Greg Rose <gvrose8192 at gmail.com>

View the changeset: https://github.com/gvrose8192/ovs-experimental/commit/07d55c8ed172

View the full build log and details: https://travis-ci.org/gvrose8192/ovs-experimental/builds/253239803?utm_source=email&utm_medium=notification


You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20170713/4091bd71/attachment.html>

More information about the build mailing list