[ovs-dev] [PATCH v3] ofproto-dpif-xlate: Fix conntrack fields on NXT_RESUME

Ben Pfaff blp at ovn.org
Wed Sep 26 23:28:32 UTC 2018


On Wed, Sep 26, 2018 at 04:15:17PM -0700, Yi-Hung Wei wrote:
> On Wed, Sep 26, 2018 at 4:07 PM Ben Pfaff <blp at ovn.org> wrote:
> >
> > On Wed, Sep 26, 2018 at 03:47:15PM -0700, Yi-Hung Wei wrote:
> > > This patch addresses the issue that the conntrack fields associated
> > > with a packet are missing after a packet is resumed by NXT_RESUME.
> > > For example, the last rule in the following OpenFlow pipeline is not
> > > working without this patch.
> > >
> > > table=0, arp,in_port=1 action=2
> > > table=0, arp,in_port=2 action=1
> > > table=0, in_port=2 icmp action=output:1
> > > table=0, in_port=1 icmp action=ct(table=1)
> > > table=1, icmp action=controller(pause) resubmit(,2)
> > > table=2, in_port=1 icmp ct_state=+trk+new action=output:2
> > >
> > > A unit test is added to prevent regression.
> > >
> > > Fixes: 77ab5fd2a95b ("Implement serializing the state of packet traversal in "continuations".")
> > > VMware-BZ: #2202764
> > > Signed-off-by: Yi-Hung Wei <yihung.wei at gmail.com>
> > > ---
> > > v1->v2: A unit test is added.
> > > v2->v3: Move the unit test from system traffic test to regular unit test.
> >
> > Thanks a lot, applied to master.
> 
> Thanks for the review. The fixed patch was in 2.6, I am not sure if
> this patch can be cleanly applied to older branch. Can we try to
> backport it at least to 2.9 ?

It applied cleanly back to 2.8 so I applied it that far.


More information about the dev mailing list