On 25 May 2017 at 13:54, Ben Pfaff <b...@ovn.org> wrote: > It's becoming more common that OSes include "ip" but not "ifconfig", so > it's best to avoid using the latter. This commit removes most references > to "ifconfig" and replaces them by "ip". It also adds a build-time check > to make it harder to introduce new uses of "ifconfig". > > Signed-off-by: Ben Pfaff <b...@ovn.org> > ---
<snip> > diff --git a/tests/test-hash.c b/tests/test-hash.c > index d1beead36ed5..f02f0218c71f 100644 > --- a/tests/test-hash.c > +++ b/tests/test-hash.c > @@ -153,14 +153,13 @@ check_hash_bytes128(void (*hash)(const void *, size_t, > uint32_t, ovs_u128 *), > OVS_PACKED(struct offset_ovs_u128 { > uint32_t a; > ovs_u128 b; > - }) in0_data; > - ovs_u128 *in0, in1; > + }) in0; > + ovs_u128 in1; > ovs_u128 out0, out1; > > - in0 = &in0_data.b; > - set_bit128(in0, i, n_bits); > set_bit128(&in1, i, n_bits); > - hash(in0, sizeof(ovs_u128), 0, &out0); > + in0.b = in1; > + hash(&in0.b, sizeof(ovs_u128), 0, &out0); > hash(&in1, sizeof(ovs_u128), 0, &out1); > if (!ovs_u128_equals(out0, out1)) { > printf("%s hash not the same for non-64 aligned data " This seems unrelated. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev