On Thu, Jul 13, 2017 at 11:30:49PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit <jpet...@ovn.org> > --- > ofproto/ofproto-dpif-rid.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/ofproto/ofproto-dpif-rid.c b/ofproto/ofproto-dpif-rid.c > index d546b150b938..26c2357007b2 100644 > --- a/ofproto/ofproto-dpif-rid.c > +++ b/ofproto/ofproto-dpif-rid.c > @@ -146,6 +146,7 @@ frozen_state_hash(const struct frozen_state *state) > hash = hash_bytes64(ALIGNED_CAST(const uint64_t *, > state->action_set), > state->action_set_len, hash); > } > + hash = hash_int(state->ofpacts_len, hash); > if (state->ofpacts_len) { > hash = hash_bytes64(ALIGNED_CAST(const uint64_t *, state->ofpacts), > state->ofpacts_len, hash);
Can you explain the benefit of this change? hash_bytes64() already uses the number of bytes hashed as one of the inputs to the hash. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev