[ovs-dev] [PATCH ovn v3] Learn the mac binding only if required

Numan Siddique nusiddiq at redhat.com
Fri Sep 27 10:05:55 UTC 2019


On Fri, Sep 27, 2019 at 11:51 AM Han Zhou <zhouhan at gmail.com> wrote:

>
>
> On Tue, Sep 24, 2019 at 1:39 PM <nusiddiq at redhat.com> wrote:
> > diff --git a/ovn-architecture.7.xml b/ovn-architecture.7.xml
> > index 6115e84b5..c98db48d2 100644
> > --- a/ovn-architecture.7.xml
> > +++ b/ovn-architecture.7.xml
> > @@ -970,6 +970,24 @@
> >              this temporary use.)
> >            </p>
> >          </dd>
> > +
> > +        <dt><code><var>R</var> = lookup_arp(<var>P</var>, <var>A</var>,
> <var>M</var>);</code></dt>
> > +        <dt><code><var>R</var> = lookup_nd(<var>P</var>, <var>A</var>,
> <var>M</var>);</code></dt>
> > +        <dd>
> > +          <p>
> > +            Implemented by storing arguments into OpenFlow fields, then
> > +            resubmitting to table 67, which <code>ovn-controller</code>
> > +            populates with flows generated from the
> <code>MAC_Binding</code>
> > +            table in the OVN Southbound database.  If there is a match
> in table
> > +            66, then its actions set the logical flow flag
> <code>MLF_LOOKUP_MAC</code>.
>
> A typo here: s/66/67
>
> Otherwise looks good to me.
>
> Acked-by: Han Zhou <hzhou8 at ebay.com>
>

Thanks for the review. I fixed by the typo and applied this patch to master.

Numan


More information about the dev mailing list