[ovs-build] Passed: horms2/ovs#203 (topic/pieter/pedit-ext-v7 - 755358e)

Travis CI builds at travis-ci.org
Sat Jan 26 10:44:42 UTC 2019


Build Update for horms2/ovs
-------------------------------------

Build: #203
Status: Passed

Duration: 1 hr, 19 mins, and 20 secs
Commit: 755358e (topic/pieter/pedit-ext-v7)
Author: Pieter Jansen van Vuuren
Message: lib/tc: make pedit mask calculations byte order agnostic

pedit allows setting entire words with an optional mask and OVS
makes use of such masks to allow setting fields that do not span
entire words.

The struct tc_pedit_key structure, which is part of the kernel
ABI, uses host byte order fields to store the mask and value for
a pedit action, however, these fields contain values in network
byte order.

In order to allow static analysis tools to check for endianness
problems this patch adds a local version of struct tc_pedit_key
which uses big endian types and refactors the relevant code as
appropriate.

In the course of making this change it became apparent that the
calculation of masks was occurring using host byte order although
the values are in network byte order. This patch also fixes that
problem by shifting values in host byte order and then converting
them to network byte order. It is believe this fixes a bug on big
endian systems although we are not in a position to test that.

Signed-off-by: Pieter Jansen van Vuuren <pieter.jansenvanvuuren at netronome.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

View the changeset: https://github.com/horms2/ovs/compare/1e1cb47418f9...755358e3ba05

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

--

You can unsubscribe from build emails from the horms2/ovs repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=20314049&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/20190126/1f74d59c/attachment-0001.html>


More information about the build mailing list