[ovs-dev] [PATCH] openflow.rst: Update to reflect current status.

Greg Rose gvrose8192 at gmail.com
Wed Jun 14 22:24:49 UTC 2017


On 06/14/2017 08:21 AM, Ben Pfaff wrote:
> OpenFlow 1.1 and 1.2 support is complete.  Simon Horman is not known to
> be working on flow entry notifications.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
>   Documentation/topics/openflow.rst | 23 +++--------------------
>   1 file changed, 3 insertions(+), 20 deletions(-)
>
> diff --git a/Documentation/topics/openflow.rst b/Documentation/topics/openflow.rst
> index 00e65411c455..6f71d6c752fc 100644
> --- a/Documentation/topics/openflow.rst
> +++ b/Documentation/topics/openflow.rst
> @@ -75,28 +75,12 @@ Here are the current approaches in a few tricky areas:
>   OpenFlow 1.1
>   ------------
>
> -The list of remaining work items for OpenFlow 1.1 is below.  It is probably
> -incomplete.
> -
> -* Match and set double-tagged VLANs (QinQ).
> -
> -  This requires kernel work for reasonable performance.
> -
> -  (optional for OF1.1+)
> -
> -* VLANs tagged with 88a8 Ethertype.
> -
> -  This requires kernel work for reasonable performance.
> -
> -  (required for OF1.1+)
> +OpenFlow 1.1 support is believed to be complete.
>
>   OpenFlow 1.2
>   ------------
>
> -OpenFlow 1.2 support requires OpenFlow 1.1 as a prerequisite. All the
> -additional work specific to Openflow 1.2 are complete.  (This is based on the
> -change log at the end of the OF1.2 spec.  I didn't compare the specs carefully
> -yet.)
> +OpenFlow 1.2 support is believed to be complete.

FWIW it looks fine to me but I am wondering about the passive voice here.

Why not just say 'OpenFlow 1.2 support is complete'?  Just curious.

>
>   OpenFlow 1.3
>   ------------
> @@ -170,8 +154,7 @@ in OVS.
>
>   * Flow entry notifications
>
> -  This seems to be modelled after OVS's NXST_FLOW_MONITOR.  (Simon Horman is
> -  working on this.)
> +  This seems to be modelled after OVS's NXST_FLOW_MONITOR.

LGTM

Acked-by: Greg Rose <gvrose8192 at gmail.com>

>
>     (EXT-187)
>     (optional for OF1.4+)
>



More information about the dev mailing list