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?
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to