Updates to some reply/request pairs
Starting to follow the convention that all set operations come in
request/reply pairs with a status code in the reply.
diff --git a/openflow_input/bsn_bw b/openflow_input/bsn_bw
index 830908f..b6e0972 100644
--- a/openflow_input/bsn_bw
+++ b/openflow_input/bsn_bw
@@ -30,7 +30,7 @@
#version any
-struct ofp_bsn_bw_enable_set {
+struct ofp_bsn_bw_enable_set_request {
uint8_t version;
uint8_t type; // OFPT_VENDOR
uint16_t length;
@@ -40,6 +40,17 @@
uint32_t enable; // 0 to disable the extension, 1 to enable it
};
+struct ofp_bsn_bw_enable_set_reply {
+ uint8_t version;
+ uint8_t type; // OFPT_VENDOR
+ uint16_t length;
+ uint32_t xid;
+ uint32_t experimenter; // OF_EXPERIMENTER_ID_BSN
+ uint32_t subtype; // BSN_BW_ENABLE_SET
+ uint32_t enable; // Resulting state, 0 disabled, 1 enabled
+ uint32_t status; // Result code: 0 success
+};
+
struct ofp_bsn_bw_enable_get_request {
uint8_t version;
uint8_t type; // OFPT_VENDOR
@@ -59,7 +70,7 @@
uint32_t enabled; // 0 if feature is disabled; 1 if feature enabled
};
-struct ofp_bsn_bw_clear_data {
+struct ofp_bsn_bw_clear_data_request {
uint8_t version;
uint8_t type; // OFPT_VENDOR
uint16_t length;
@@ -67,3 +78,13 @@
uint32_t experimenter; // OF_EXPERIMENTER_ID_BSN
uint32_t subtype; // BSN_BW_CLEAR_DATA
};
+
+struct ofp_bsn_bw_clear_data_reply {
+ uint8_t version;
+ uint8_t type; // OFPT_VENDOR
+ uint16_t length;
+ uint32_t xid;
+ uint32_t experimenter; // OF_EXPERIMENTER_ID_BSN
+ uint32_t subtype; // BSN_BW_CLEAR_DATA
+ uint32_t status; // Result code, 0 success
+};