[ovs-dev] [PATCH] OPENFLOW-1.1+: Remove completed tasks.

[Simon Horman]

On Thu, Aug 29, 2013 at 09:52:07PM -0700, Ben Pfaff wrote:
> It appears to me that OFPST_TABLE is correct for OF1.1+, so I'm removing
> that.
> 
> DESIGN describes OF1.2 flow_mod, so I'm removing that.
> 
> Port and queue stats include duration these days.  The proposed patch to
> implement groups includes duration also.
> 
> Signed-off-by: Ben Pfaff <blp at nicira.com>


Reviewed-by: Simon Horman <horms at verge.net.au>

> ---
>  OPENFLOW-1.1+ |   18 ------------------
>  1 file changed, 18 deletions(-)
> 
> diff --git a/OPENFLOW-1.1+ b/OPENFLOW-1.1+
> index 753dbba..aacbca9 100644
> --- a/OPENFLOW-1.1+
> +++ b/OPENFLOW-1.1+
> @@ -69,18 +69,6 @@ probably incomplete.
>        behavior does not change.
>        [required for OF1.1 and OF1.2]
>  
> -    * Flow table stats (OFPST_TABLE).
> -
> -        * Reference count (active entries)
> -          [implemented]
> -          [required for OF1.1 and OF1.2]
> -
> -        * Packet Lookups
> -          [required for OF1.1; optional for OF1.2]
> -
> -        * Packet Matches
> -          [required for OF1.1; optional for OF1.2]
> -
>      * MPLS.  Simon Horman maintains a patch series that adds this
>        feature.  This is partially merged.
>        [optional for OF1.1+]
> @@ -122,9 +110,6 @@ end of the OF1.2 spec.  I didn't compare the specs carefully yet.)
>          * Add ability to delete flow in all tables.
>            [required for OF1.2+]
>  
> -        * Update DESIGN to describe OF1.2 behavior also.
> -          [required for OF1.2+]
> -
>  OpenFlow 1.3
>  ------------
>  
> @@ -187,9 +172,6 @@ didn't compare the specs carefully yet.)
>        optimization in some cases for the software switch.
>        [optional for OF1.3+]
>  
> -    * Duration Statistics
> -      - New for Per Port, Per Queue, Per Group
> -      [required for v1.3+]
>  
>  How to contribute
>  -----------------
> -- 
> 1.7.10.4
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>