On Thu, Sep 06, 2018 at 04:19:51PM -0700, Yi-Hung Wei wrote: > On Thu, Sep 6, 2018 at 4:07 PM Ben Pfaff <[email protected]> wrote: > > > > On Thu, Sep 06, 2018 at 09:56:26AM -0700, Yi-Hung Wei wrote: > > > On Thu, Sep 6, 2018 at 6:18 AM Tiago Lam <[email protected]> wrote: > > > > > > > > Commit 3f1087c added a per zone limit test which relied on the > > > > CHECK_CT_DPIF_FLUSH_BY_CT_TUPLE m4 macro to skip the test when executing > > > > in a userspace datapath (since the per zone limit feature is not yet > > > > implemented in userspace). That macro, however, has been removed in > > > > commit 271e48a ("conntrack: Support conntrack flush by ct 5-tuple") > > > > which was causing the test to fail when executing in userspace. > > > > > > > > Instead, a new m4 macro, CHECK_CT_DPIF_PER_ZONE_LIMIT, is introduced to > > > > make the same differentiation, until userspace doesn't support the per > > > > zone limit. > > > > > > > > CC: Yi-Hung Wei <[email protected]> > > > > Fixes: 3f1087c ("system-traffic: Add conntrack per zone limit test > > > > case") > > > > Signed-off-by: Tiago Lam <[email protected]> > > > > --- > > > Thanks Tiago for the fix. I will work on the userspace zone limitation > > > later on. > > > > > > Acked-by: Yi-Hung Wei <[email protected]> > > > > Thanks Tiago and Yi-Hung. I applied this to master. Let me know if I > > should backport it. > > Hi Ben, > > Please help to backport it to branch-2.10.
Done. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
