Thanks Ben I had a quick look; I will test out today and get back.
Darrell On 6/7/17, 9:37 AM, "[email protected] on behalf of Ben Pfaff" <[email protected] on behalf of [email protected]> wrote: On Sat, Jun 03, 2017 at 01:38:14PM -0700, Darrell Ball wrote: > Part of the hash input for nat_range_hash() was accidently > truncated, so recover it here. Also update the local variable > name to better reflect the intention and add missing comments. > > Signed-off-by: Darrell Ball <[email protected]> Thanks a lot for the improvement. I'm sorry I screwed this up on review. I feel like this could benefit from a little more abstraction. I sent out a 2-patch series for you to take a look at: https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_pipermail_ovs-2Ddev_2017-2DJune_333573.html&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=DYoxK1j8Y-F_qsiBKKfBPXvMiD5Sso-WINF5q-UvqBI&s=Cn4cLI5MDgHTEtgx9-5DB09ZDuY-7TbYfnUwOGUnZl4&e= https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_pipermail_ovs-2Ddev_2017-2DJune_333574.html&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=DYoxK1j8Y-F_qsiBKKfBPXvMiD5Sso-WINF5q-UvqBI&s=1vffunQcO8aXlL2OD08qZUjFv9UUz7-bisYGGGMMYM4&e= _______________________________________________ dev mailing list [email protected] https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=DYoxK1j8Y-F_qsiBKKfBPXvMiD5Sso-WINF5q-UvqBI&s=J7BbzKyPMkSADLePw4ZFt9U2aw2XRCRYUEAAIlRZqf4&e= _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
