Thanks, I pushed this to master, branch-1.3, and branch-1.2.
On Tue, Oct 18, 2011 at 11:20:47AM -0700, Ethan Jackson wrote: > 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
