[ovs-build] Passed: ovsrobot/ovs#2691 (series_184667 - b9e2498)

Travis CI builds at travis-ci.org
Fri Jun 19 21:12:56 UTC 2020


Build Update for ovsrobot/ovs
-------------------------------------

Build: #2691
Status: Passed

Duration: 27 mins and 59 secs
Commit: b9e2498 (series_184667)
Author: Satish Dhote
Message: tc: Changes to netlink message population for OVS-TC Flower Offload

OVS-TC offload sends only fields that are not completely masked out
to the TC kernel module. Some of the hardware we work with requires
all fields extracted from the packet along with the mask to be sent
to the driver. Hardware may optimize tables based on entire fields
of packet and mask of fields, even though certain fields are masked
out.

OVS when offloading to ovs kernel datapath sends the entire key
along with the mask. Proposal requests for similar behaviour in
ovs-vswitchd in case of tc-offload. While constructing netlink
messages, ovs-vswitchd is removing completely masked out fields.
Sending additional masked fields should not impact existing drivers.
TC internally uses dissectors to extract non masked fields, so it
should not impact existing drivers in tc.

Example:
ovs-rule:ovs-ofctl add-flow br0 \
    "table=0, dl_dst=00:a1:45:23:23:11/ff:ff:ff:ff:ff:ff, actions=1"

If we have above rule in ovs and an icmp packet matching destination
MAC 00:a1:45:23:23:11 arrives, ovs prepares the key and mask with all
relevant fields. When sending to TC it removes fields which are
completely masked out. In this case ethertype, and dst_mac fields are
sent as part of netlink message to tc.

OVS patch also requires minor fixes in TC. Patch to TC is submitted
at following link. 1/3 patch has the fix for the same.
http://patchwork.ozlabs.org/project/netdev/list/?series=184493

Co-authored-by: Chandra Kesava <kesavac at gmail.com>
Co-authored-by: Prathibha Nagooru <prathibha.nagooru at oneconvergence.com>
Signed-off-by: Chandra Kesava <kesavac at gmail.com>
Signed-off-by: Prathibha Nagooru <prathibha.nagooru at oneconvergence.com>
Signed-off-by: Satish Dhote <satish.d at oneconvergence.com>
Signed-off-by: 0-day Robot <robot at bytheb.org>

View the changeset: https://github.com/ovsrobot/ovs/commit/b9e249845fb3

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

--

You can unsubscribe from build emails from the ovsrobot/ovs repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=22285853&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/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...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20200619/47692e59/attachment.html>


More information about the build mailing list