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 <b...@nicira.com>
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > Signed-off-by: Simon Horman <ho...@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.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to