[ovs-dev] [PATCH 1/2] ovn: Minor cleanups in "physical" component comments.

[Gal Sagie]

On Wed, Jul 15, 2015 at 3:33 AM, Justin Pettit <jpettit at nicira.com> wrote:

> There's only this one patch in this "series".  I forgot to clean that up
> before I sent it.
>
> --Justin
>
>
> > On Jul 14, 2015, at 5:31 PM, Justin Pettit <jpettit at nicira.com> wrote:
> >
> > Signed-off-by: Justin Pettit <jpettit at nicira.com>
> > ---
> > ovn/controller/physical.c |   11 ++++++-----
> > 1 files changed, 6 insertions(+), 5 deletions(-)
> >
> > diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c
> > index dce0c2f..9e7f2f4 100644
> > --- a/ovn/controller/physical.c
> > +++ b/ovn/controller/physical.c
> > @@ -92,7 +92,7 @@ physical_run(struct controller_ctx *ctx)
> >      * 64 for logical-to-physical translation. */
> >     const struct sbrec_binding *binding;
> >     SBREC_BINDING_FOR_EACH (binding, ctx->ovnsb_idl) {
> > -        /* Find the Openflow port for the logical port, as 'ofport'.
> If it's
> > +        /* Find the OpenFlow port for the logical port, as 'ofport'.
> If it's
> >          * on a remote chassis, this is the OpenFlow port for the tunnel
> to
> >          * that chassis (and set 'local' to false).  Otherwise, if it's
> on the
> >          * chassis we're managing, this is the OpenFlow port for the vif
> itself
> > @@ -134,13 +134,14 @@ physical_run(struct controller_ctx *ctx)
> >
> >         struct match match;
> >         if (local) {
> > -            /*
> > -             * Packets that arrive from a vif can belong to a VM or
> > +            /* Packets that arrive from a vif can belong to a VM or
> >              * to a container located inside that VM. Packets that
> >              * arrive from containers have a tag (vlan) associated with
> them.
> > -             *
> > -             * Table 0, Priority 150 and 100.
> > +             */
> > +
> > +            /* Table 0, Priority 150 and 100.
> >              * ==============================
> > +             *
> >              * Priority 150 is for traffic belonging to containers. For
> such
> >              * traffic, match on the tags and then strip the tag.
> >              * Priority 100 is for traffic belonging to VMs.
> > --
> > 1.7.5.4
> >
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>


Acked-by: Gal Sagie <gal.sagie at gmail.com>

-- 
Best Regards ,

The G.