Looks good. Ethan
On Sun, Oct 9, 2011 at 15:52, Ben Pfaff <[email protected]> wrote: > This makes OFPAT_ENQUEUE consistent with OFPAT_OUTPUT for the purpose of > sending a packet back out the input port: both only do it if the port is > given as OFPP_IN_PORT. > > Found by inspection. > --- > ofproto/ofproto-dpif.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c > index 9e8b1fb..915e64f 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -3355,6 +3355,8 @@ xlate_enqueue_action(struct action_xlate_ctx *ctx, > ofp_port = ntohs(oae->port); > if (ofp_port == OFPP_IN_PORT) { > ofp_port = ctx->flow.in_port; > + } else if (ofp_port == ctx->flow.in_port) { > + return; > } > odp_port = ofp_port_to_odp_port(ofp_port); > > -- > 1.7.2.5 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
