On Wed, Jul 26, 2017 at 12:06 AM, Ilya Maximets <[email protected]> wrote: > On 25.07.2017 21:39, Andy Zhou wrote: >> Commit 42781e77035d (bond: Unify hash functions in hash action and entry >> lookup.) changed the BM_TCP's hash function, but did not update >> hash mask fields accordingly. Found by inspection. >> >> CC: Ilya Maximets <[email protected]> >> Signed-off-by: Andy Zhou <[email protected]> >> --- > > Good catch. I forget about this. > One minor comment below, but it's not important. > > Acked-by: Ilya Maximets <[email protected]>
Thanks for the review. Pushed. >> + flow_mask_hash_fields(flow, wc, >> + NX_HASH_FIELDS_SYMMETRIC_L3L4_UDP); > > Above function call, actually, fits in 79 characters limit and can be > written in one line. > You are right. Thanks. I merged those two lines before pushing. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
