[ovs-git] [openvswitch/ovs] f7158f: travis: Remove sparse support.
GitHub
noreply at github.com
Tue Oct 16 22:12:49 UTC 2018
Branch: refs/heads/branch-2.5
Home: https://github.com/openvswitch/ovs
Commit: f7158faf655266e4ab2ac20c158a826df56d5011
https://github.com/openvswitch/ovs/commit/f7158faf655266e4ab2ac20c158a826df56d5011
Author: Ben Pfaff <blp at ovn.org>
Date: 2018-10-16 (Tue, 16 Oct 2018)
Changed paths:
M .travis/build.sh
M .travis/prepare.sh
Log Message:
-----------
travis: Remove sparse support.
"sparse" failed to build with this old branch, see e.g.
https://travis-ci.org/openvswitch/ovs/jobs/436851158
Signed-off-by: Ben Pfaff <blp at ovn.org>
Commit: ea74b10c0838f156a98bac8c43ee79628b8fc0ce
https://github.com/openvswitch/ovs/commit/ea74b10c0838f156a98bac8c43ee79628b8fc0ce
Author: Ben Pfaff <blp at ovn.org>
Date: 2018-10-16 (Tue, 16 Oct 2018)
Changed paths:
M lib/odp-util.c
M lib/packets.h
M ofproto/ofproto-dpif-sflow.c
Log Message:
-----------
odp-execute: Fix broken build with Clang as compiler.
Builds of branch-2.7 have been failing on Travis when Clang is used as
compiler due to:
../ofproto/ofproto-dpif.c:2057:46: warning: taking address of packed member
'eth_src' of class or structure 'eth_header' may result in an unaligned
pointer value [-Waddress-of-packed-member]
netdev_get_etheraddr(ofport->up.netdev, ð->eth_src);
^~~~~~~~~~~~
../ofproto/ofproto-dpif.c:2082:50: warning: taking address of packed member
'eth_src' of class or structure 'eth_header' may result in an unaligned
pointer value [-Waddress-of-packed-member]
netdev_get_etheraddr(ofport->up.netdev, ð->eth_src);
^~~~~~~~~~~~
On master these don't come up because of commit 1620b7ea68c2 ("packets:
Remove unnecessary "packed" annotations."), which removed the packed
annotation that causes the warning. This commit applies enough of that
commit to make the build pass.
Signed-off-by: Ben Pfaff <blp at ovn.org>
Commit: 84b0df9ab18c381e53040eb64c14edf992b6ddeb
https://github.com/openvswitch/ovs/commit/84b0df9ab18c381e53040eb64c14edf992b6ddeb
Author: Joe Stringer <joe at ovn.org>
Date: 2018-10-16 (Tue, 16 Oct 2018)
Changed paths:
M tests/test-hash.c
Log Message:
-----------
test-hash: Fix unaligned pointer value error.
Clang 4.0 complains:
../tests/test-hash.c:160:16: error: taking address of packed member 'b' of
class or structure 'offset_ovs_u128' may result in an unaligned pointer value
[-Werror,-Waddress-of-packed-member]
in0 = &in0_data.b;
Set the bit in the aligned u128 first then copy the contents into the
offset u128 so that we don't have to take the address of the non-aligned
u128 and pass it to set_bit128.
For the 256byte_hash, fix it up so that it's actually testing the 256B
hash inside a 32-bit offset u128 as well.
Suggested-by: Ben Pfaff <blp at ovn.org>
Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>
Compare: https://github.com/openvswitch/ovs/compare/a598e6ae93f5...84b0df9ab18c
**NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
More information about the git
mailing list