[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