[ovs-dev] [PATCH] ofproto-dpif: Make OFPAT_ENQUEUE to input port do nothing.

Ben Pfaff blp at nicira.com
Tue Oct 18 20:50:50 UTC 2011


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 <blp at nicira.com> 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
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> >



More information about the dev mailing list