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. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
