On Wed, Apr 24, 2019 at 03:13:22PM -0700, Yi-Hung Wei wrote:
> On Wed, Apr 24, 2019 at 9:42 AM Ben Pfaff <b...@ovn.org> wrote:
> >
> > The commits that implemented these features forgot to update the
> > documentation.
> >
> > Signed-off-by: Ben Pfaff <b...@ovn.org>
> > ---
> >  Documentation/topics/openflow.rst | 26 +-------------------------
> >  1 file changed, 1 insertion(+), 25 deletions(-)
> >
> > diff --git a/Documentation/topics/openflow.rst 
> > b/Documentation/topics/openflow.rst
> > index bb7559047742..b6a5c6b54920 100644
> > --- a/Documentation/topics/openflow.rst
> > +++ b/Documentation/topics/openflow.rst
> > @@ -90,12 +90,6 @@ following additional work.  (This is based on the change 
> > log at the end of the
> >  OF1.3 spec, reusing most of the section titles directly.  I didn't compare 
> > the
> >  specs carefully yet.)
> >
> > -* Add support for multipart requests.
> > -
> > -  Currently we always report ``OFPBRC_MULTIPART_BUFFER_OVERFLOW``.
> > -
> > -  (optional for OF1.3+)
> > -
> >  * IPv6 extension header handling support.
> >
> >    Fully implementing this requires kernel support.  This likely will take 
> > some
> > @@ -195,17 +189,6 @@ OpenFlow 1.4 only
> >  Those features are those only available in OpenFlow 1.4, other OpenFlow 1.4
> >  features are listed in the previous section.
> >
> > -* More extensible wire protocol
> > -
> > -  Many on-wire structures got TLVs.
> > -
> > -  All required features are now supported.
> > -  Remaining optional: table desc, table-status
> > -
> > -  (EXT-262)
> > -
> > -  (required for OF1.4+)
> > -
> >  * Optical port properties
> >
> >    (EXT-154)
> > @@ -237,8 +220,7 @@ OpenFlow 1.5 only
> >  -----------------
> >
> >  Those features are those only available in OpenFlow 1.5, other OpenFlow 1.5
> > -features are listed in the previous section.  Note that this list is not
> > -definitive as OpenFlow 1.5 is not yet published.
> > +features are listed in the previous section.
> >
> >  * Egress Tables
> >
> > @@ -246,12 +228,6 @@ definitive as OpenFlow 1.5 is not yet published.
> >
> >    (optional for OF1.5+)
> >
> > -* Extensible Flow Entry Statistics
> > -
> > -  (EXT-334)
> > -
> > -  (required for OF1.5+)
> > -
> >  * Flow Entry Statistics Trigger
> >
> >    (EXT-335)
> > --
> I did some quick checks.  All the aforementioned features (multipart
> requests, table desc, table-status TLV, Extensible Flow Entry
> Statistics) are in OVS already.
> 
> Thanks for updating the doc.
> 
> Acked-by: Yi-Hung Wei <yihung....@gmail.com>

Thank you for checking, and for the review.  I applied this to master.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to