On Wed, Sep 26, 2018 at 4:07 PM Ben Pfaff <b...@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....@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 ?

Thanks,

-Yi-Hung
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to