commit e815848efa7e95eb5ea82ea343d3cfe7ea645da2
author Vishnu Emmadi <vishnu.emmadi@bigswitch.com> Mon Sep 30 18:25:44 2013 -0700
committer Vishnu Emmadi <vishnu.emmadi@bigswitch.com> Mon Sep 30 18:25:44 2013 -0700
tree 1332bbe12bc200d2a019fa64a67b8ccedf91cc28
parent fcb9023eb1de5d7b26011e90e2e87097a9bd6e10

address review comments

also removed data from pdu timeout notification

openflow_input/bsn_pdu

diff --git a/openflow_input/bsn_pdu b/openflow_input/bsn_pdu
index 505f8fb..c2735de 100644
--- a/openflow_input/bsn_pdu
+++ b/openflow_input/bsn_pdu
@@ -30,19 +30,19 @@
 
 #version any
 
-struct of_bsn_set_port_pdu_tx_info_request : of_bsn_header {
+struct of_bsn_pdu_tx_request : of_bsn_header {
     uint8_t version;
     uint8_t type == 4;
     uint16_t length;
     uint32_t xid;
     uint32_t experimenter == 0x5c16c7;
     uint32_t subtype == 31;
-    uint32_t pdu_tx_interval_ms;
+    uint32_t tx_interval_ms;
     of_port_no_t port_no;
-    of_octets_t pdu_payload;
+    of_octets_t data;
 };
 
-struct of_bsn_set_port_pdu_tx_info_reply : of_bsn_header {
+struct of_bsn_pdu_tx_reply : of_bsn_header {
     uint8_t version;
     uint8_t type == 4;
     uint16_t length;
@@ -52,20 +52,19 @@
     uint32_t status; // 0 means success
 };
 
-struct of_bsn_set_port_pdu_expected_rx_info_request : of_bsn_header {
+struct of_bsn_pdu_rx_request : of_bsn_header {
     uint8_t version;
     uint8_t type == 4;
     uint16_t length;
     uint32_t xid;
     uint32_t experimenter == 0x5c16c7;
     uint32_t subtype == 33;
+    uint32_t timeout_ms;
     of_port_no_t port_no;
-    uint16_t is_pdu_expected; // following fields are valid only if this is 1
-    uint32_t pdu_timeout_ms;
-    of_octets_t pdu_payload;
+    of_octets_t data;
 };
 
-struct of_bsn_set_port_pdu_expected_rx_info_reply : of_bsn_header {
+struct of_bsn_pdu_rx_reply : of_bsn_header {
     uint8_t version;
     uint8_t type == 4;
     uint16_t length;
@@ -75,7 +74,7 @@
     uint32_t status; // 0 means success
 };
 
-struct of_bsn_pdu_timedout_notif : of_bsn_header {
+struct of_bsn_pdu_rx_timeout : of_bsn_header {
     uint8_t version;
     uint8_t type == 4;
     uint16_t length;
@@ -83,5 +82,4 @@
     uint32_t experimenter == 0x5c16c7;
     uint32_t subtype == 35;
     of_port_no_t port_no;
-    of_octets_t expected_pdu_payload;
 };