On Tue, Sep 11, 2018 at 11:59:36PM -0700, Martin Xu wrote: > Add a symmetric_l3 hash method that uses both network destination > address and network source address. > > VMware-BZ: #2112940 > > Signed-off-by: Martin Xu <[email protected]> > CC: Ben Pfaff <[email protected]>
Thanks for the patch. It looks OK to me. I have a few comments. I think that this should be mentioned in NEWS. In ovs-ofctl.8.in, I don't think "Network" needs to be capitalized, in two places. In the tests directory, grepping for "symmetric", I see a number of places where we test related features. It would be wise to add similar tests for the new hashing method. Thanks, Ben. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
