Hi,

About 10 days ago I sent this mail.
Please check attached patch.

thanks,

On 2015年09月18日 15:58, Minoru TAKAHASHI wrote:
> Hi,
> 
> I found a bug about ofp-msgs.h.
> 
> Number of OFPMP_TABLE_DESC is *14*.
> (Plese refer to "Header file openflow.h" of OF1.4/5spec)
> 
> However, ofp-msgs.h set these to *15*.
> https://github.com/openvswitch/ovs/blob/master/lib/ofp-msgs.h#L381
> https://github.com/openvswitch/ovs/blob/master/lib/ofp-msgs.h#L384
> 
> Attached patch fixes this probelm.
> 
> thanks,
> 
>From 43176acaed08afcc060474cd3c3b7fba850ff5c0 Mon Sep 17 00:00:00 2001
From: Minoru TAKAHASHI <takahashi.mino...@gmail.com>
Date: Fri, 18 Sep 2015 15:38:10 +0900
Subject: [PATCH] ofp-msgs: fix enum of OFPMP_TABLE_DESC

Signed-off-by: Minoru TAKAHASHI <takahashi.mino...@gmail.com>
---
 lib/ofp-msgs.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/ofp-msgs.h b/lib/ofp-msgs.h
index 132e405..bce5283 100644
--- a/lib/ofp-msgs.h
+++ b/lib/ofp-msgs.h
@@ -378,10 +378,10 @@ enum ofpraw {
     /* OFPST 1.3+ (12): struct ofp13_table_features, uint8_t[8][]. */
     OFPRAW_OFPST13_TABLE_FEATURES_REPLY,
 
-    /* OFPST 1.4+ (15): void. */
+    /* OFPST 1.4+ (14): void. */
     OFPRAW_OFPST14_TABLE_DESC_REQUEST,
 
-    /* OFPST 1.4+ (15): struct ofp14_table_desc, uint8_t[8][]. */
+    /* OFPST 1.4+ (14): struct ofp14_table_desc, uint8_t[8][]. */
     OFPRAW_OFPST14_TABLE_DESC_REPLY,
 
     /* OFPST 1.0-1.4 (13): void. */
-- 
1.9.1

_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to