On 09/01/2015 08:21 PM, Justin Pettit wrote:
> Signed-off-by: Justin Pettit <jpet...@nicira.com>

some wording suggestions inline, but otherwise:

Acked-by: Russell Bryant <rbry...@redhat.com>

> ---
>  ovn/ovn-sb.xml |   43 +++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 43 insertions(+), 0 deletions(-)
> 
> diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
> index 38837c1..cbae2e9 100644
> --- a/ovn/ovn-sb.xml
> +++ b/ovn/ovn-sb.xml
> @@ -321,6 +321,49 @@
>        the logical output port replaced by the group member.
>      </p>
>  
> +    <p><em>Pipeline Stages</em></p>
> +
> +    <p>
> +      <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

> +      detail and subject to change.  This section describes the current
> +      logical flow table.
> +    </p>
> +
> +    <p>
> +      The ingress pipeline consists of the following stages:
> +    </p>
> +    <ul>
> +      <li>
> +        Port Security (Table 0): Validates the source address, drops
> +        packets with a VLAN tag, and, if configured, verifies that the
> +        logical port is allowed to send with the source address.
> +      </li>
> +
> +      <li>
> +        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".

> +        flooded to all ports in the logical switch.
> +      </li>
> +    </ul>
> +
> +    <p>
> +      The egress pipeline consists of the following stages:
> +    </p>
> +    <ul>
> +      <li>
> +        ACL (Table 0): Applies any specified access control lists.
> +      </li>
> +
> +      <li>
> +        Port Security (Table 1): If configured, verifies that the
> +        logical port is allowed to receive packets with the destination
> +        address.
> +      </li>
> +    </ul>
> +
>      <column name="logical_datapath">
>        The logical datapath to which the logical flow belongs.
>      </column>
> 


-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to