[ovs-dev] [PATCH v7 1/3] ofproto-dpif: MPLS recirculation

[Simon Horman]

On Mon, Jun 23, 2014 at 03:58:28PM -0700, Ben Pfaff wrote:
> On Mon, Jun 23, 2014 at 3:55 PM, Simon Horman <horms at verge.net.au> wrote:
> > On Mon, Jun 23, 2014 at 02:50:20PM -0700, Ben Pfaff wrote:
> >> On Mon, Jun 16, 2014 at 10:09:38AM +0900, Simon Horman wrote:
> >> > In some cases an pop MPLS action changes a packet to be a non-mpls packet.
> >> > In this case subsequent any L3+ actions require access to portions
> >> > of the packet which were not decoded as they were opaque when the
> >> > packet was MPLS. Allow such actions to be translated by
> >> > first recirculating the packet.
> >> >
> >> > Co-authored-by: Ben Pfaff <blp at nicira.com>
> >> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> >> > Signed-off-by: Simon Horman <horms at verge.net.au>
> >>
> >> I'm pretty happy with this.
> >>
> >> One final question: I think that compose_recirculate_action() needs to
> >> set ctx->exit unconditionally, not just in the failure cases.  Do you agree?
> >
> > Yes I agree.
> >
> > Let me know if you would like me to re-spin.
> 
> Please. That would save me a minute or two. Thanks!

Done :)