[ovs-dev] [PATCH 3/3] ovn-sb: Describe the pipeline stages generated by ovn-northd.

Justin Pettit jpettit at nicira.com
Wed Sep 2 01:02:30 UTC 2015


> On Sep 1, 2015, at 5:37 PM, Russell Bryant <rbryant at redhat.com> wrote:
> 
> On 09/01/2015 08:21 PM, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit <jpettit at nicira.com>
>> 
>> +      <code>ovn-northd</code> is responsible for populating the
>> +      <ref table="Logical_Flow"/>, so the stages are an implementation
> 
> Should you add "table" here?
> 
>  the <ref table="Logical_Flow"/> table

Done.  Thanks.

>> +        L2 Destination Lookup (Table 1): Forwards known unicast
>> +        addresses to the appropriate logical port.  Unicast packets to
>> +        unknown hosts are forwarded to logical ports configured to have
>> +        an <code>unknown</code> addresses.  Broadcast, and multicast are
> 
> "an unknown addresses" would read better as "an unknown set of
> addresses" or just "unknown addresses".

The <code> block was trying to indicate the special value "unknown" in the "mac" column, not emphasize it.  How about this instead?

        Unicast packets to unknown hosts are forwarded to
        logical ports configured with the special <code>unknown</code> 
        mac address. 

--Justin





More information about the dev mailing list