diff options
author | Justin Pettit <jpettit@ovn.org> | 2016-05-03 18:20:51 -0700 |
---|---|---|
committer | Justin Pettit <jpettit@ovn.org> | 2016-05-08 09:26:19 -0700 |
commit | 2ff8484bbf88ce2552f0a55c6032d30eaff377e6 (patch) | |
tree | cc86a6270076eed91a2b1b49b71cc1421818e062 /tests | |
parent | 20322d4b6ef2c4513535c7837a1e8becae1da7f1 (diff) | |
download | openvswitch-2ff8484bbf88ce2552f0a55c6032d30eaff377e6.tar.gz |
util: Pass 128-bit arguments directly instead of using pointers.
Commit f2d105b5 (ofproto-dpif-xlate: xlate ct_{mark, label} correctly.)
introduced the ovs_u128_and() function. It directly takes ovs_u128
values as arguments instead of pointers to them. As this is a bit more
direct way to deal with 128-bit values, modify the other utility
functions to do the same.
Signed-off-by: Justin Pettit <jpettit@ovn.org>
Acked-by: Joe Stringer <joe@ovn.org>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/test-hash.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/test-hash.c b/tests/test-hash.c index 67a1f6c89..d1beead36 100644 --- a/tests/test-hash.c +++ b/tests/test-hash.c @@ -162,7 +162,7 @@ check_hash_bytes128(void (*hash)(const void *, size_t, uint32_t, ovs_u128 *), set_bit128(&in1, i, n_bits); hash(in0, sizeof(ovs_u128), 0, &out0); hash(&in1, sizeof(ovs_u128), 0, &out1); - if (!ovs_u128_equals(&out0, &out1)) { + if (!ovs_u128_equals(out0, out1)) { printf("%s hash not the same for non-64 aligned data " "%016"PRIx64"%016"PRIx64" != %016"PRIx64"%016"PRIx64"\n", name, out0.u64.lo, out0.u64.hi, out1.u64.lo, out1.u64.hi); @@ -214,7 +214,7 @@ check_256byte_hash(void (*hash)(const void *, size_t, uint32_t, ovs_u128 *), set_bit128(in1, i, n_bits); hash(in0, sizeof(ovs_u128) * 16, 0, &out0); hash(in1, sizeof(ovs_u128) * 16, 0, &out1); - if (!ovs_u128_equals(&out0, &out1)) { + if (!ovs_u128_equals(out0, out1)) { printf("%s hash not the same for non-64 aligned data " "%016"PRIx64"%016"PRIx64" != %016"PRIx64"%016"PRIx64"\n", name, out0.u64.lo, out0.u64.hi, out1.u64.lo, out1.u64.hi); |