diff mbox

[ovs-dev] Remove duplicate description about Experimenter classes

Message ID 1502175305-116630-1-git-send-email-yi.y.yang@intel.com
State Accepted
Headers show

Commit Message

Yang, Yi Aug. 8, 2017, 6:55 a.m. UTC 
commit 3d2fbd70bda514f7327970b859663f34f994290c brought
duplicate description about Experimenter classes
ONFOXM_ET and NXOXM_NSH in lib/meta-flow.xml, branch-2.8
has the same issue.

Signed-off-by: Yi Yang <yi.y.yang@intel.com>
---
 lib/meta-flow.xml | 23 ++---------------------
 1 file changed, 2 insertions(+), 21 deletions(-)

Comments

Ben Pfaff Aug. 9, 2017, 8 p.m. UTC | #1 
On Tue, Aug 08, 2017 at 02:55:05PM +0800, Yi Yang wrote:
> commit 3d2fbd70bda514f7327970b859663f34f994290c brought
> duplicate description about Experimenter classes
> ONFOXM_ET and NXOXM_NSH in lib/meta-flow.xml, branch-2.8
> has the same issue.
> 
> Signed-off-by: Yi Yang <yi.y.yang@intel.com>

Thanks, applied to master and branch-2.8.
diff mbox

Patch

diff --git a/lib/meta-flow.xml b/lib/meta-flow.xml
index 3066c67..065fb03 100644
--- a/lib/meta-flow.xml
+++ b/lib/meta-flow.xml
@@ -721,27 +721,8 @@  tcp,tp_src=0x07c0/0xfff0
   <dl>
     <dt>0x4f4e4600 (<code>ONFOXM_ET</code>)</dt>
     <dd>
-      Used by official Open Networking Foundation extensions to OpenFlow 1.3 in
-      e.g. [TCP Flags Match Field Extension].
-    </dd>
-
-    <dt>0x005ad650 (<code>NXOXM_NSH</code>)</dt>
-    <dd>
-      Used by Open vSwitch for NSH extensions, in the absence of an official
-      ONF-assigned class.  (This OUI is randomly generated.)
-    </dd>
-  </dl>
-
-  <p>
-    OpenFlow says that support for experimenter fields is optional.  Open
-    vSwitch 2.4 and later does support them, so that it can support the
-    following experimenter classes:
-  </p>
-
-  <dl>
-    <dt>0x4f4e4600 (<code>ONFOXM_ET</code>)</dt>
-    <dd>
-      Used by official Open Networking Foundation extensions to OpenFlow 1.3 in
+      Used by official Open Networking Foundation extensions in OpenFlow 1.3
+      and later.
       e.g. [TCP Flags Match Field Extension].
     </dd>