[ovs-dev] [PATCH] system-traffic: Fix conntrack per zone limit test.

Yi-Hung Wei yihung.wei at gmail.com
Thu Sep 6 23:19:51 UTC 2018


On Thu, Sep 6, 2018 at 4:07 PM Ben Pfaff <blp at ovn.org> 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 <tiago.lam at intel.com> 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 <yihung.wei at gmail.com>
> > > Fixes: 3f1087c ("system-traffic: Add conntrack per zone limit test case")
> > > Signed-off-by: Tiago Lam <tiago.lam at intel.com>
> > > ---
> > Thanks Tiago for the fix. I will work on the userspace zone limitation later on.
> >
> > Acked-by: Yi-Hung Wei <yihung.wei at gmail.com>
>
> 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.

Thanks,

-Yi-Hung


More information about the dev mailing list