aboutsummaryrefslogtreecommitdiffstats
path: root/src/vnet
diff options
context:
space:
mode:
authorJakub Grajciar <jgrajcia@cisco.com>2019-08-26 12:55:15 +0200
committerOle Trøan <otroan@employees.org>2019-09-11 15:14:02 +0000
commit3d1ef873da900ddf4ef18d5cf7709aab70ed6b7d (patch)
treef1fa77818a9a67e2f149ba540bcb251dc9f4f10b /src/vnet
parent558e3e09577a7b49e2fec58e8ac27f3f3ae0592f (diff)
bonding: API cleanup
Use consistent API types. Type: fix Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com> Change-Id: Idbba4ab6a412b75338e3149e51476693f0862f16 Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com>
Diffstat (limited to 'src/vnet')
-rw-r--r--src/vnet/bonding/bond.api74
-rw-r--r--src/vnet/bonding/bond_api.c11
2 files changed, 54 insertions, 31 deletions
diff --git a/src/vnet/bonding/bond.api b/src/vnet/bonding/bond.api
index 5d9a0563c1a..34fa0204a39 100644
--- a/src/vnet/bonding/bond.api
+++ b/src/vnet/bonding/bond.api
@@ -19,7 +19,29 @@
the bonding device driver
*/
-option version = "1.0.2";
+option version = "2.0.0";
+
+import "vnet/interface_types.api";
+import "vnet/ethernet/ethernet_types.api";
+
+enum bond_mode
+{
+ BOND_API_MODE_ROUND_ROBIN = 1,
+ BOND_API_MODE_ACTIVE_BACKUP = 2,
+ BOND_API_MODE_XOR = 3,
+ BOND_API_MODE_BROADCAST = 4,
+ BOND_API_MODE_LACP = 5,
+};
+
+enum bond_lb_algo
+{
+ BOND_API_LB_ALGO_L2 = 0,
+ BOND_API_LB_ALGO_L34 = 1,
+ BOND_API_LB_ALGO_L23 = 2,
+ BOND_API_LB_ALGO_RR = 3,
+ BOND_API_LB_ALGO_BC = 4,
+ BOND_API_LB_ALGO_AB = 5,
+};
/** \brief Initialize a new bond interface with the given paramters
@param client_index - opaque cookie to identify the sender
@@ -36,11 +58,11 @@ define bond_create
u32 client_index;
u32 context;
u32 id;
- u8 use_custom_mac;
- u8 mac_address[6];
- u8 mode;
- u8 lb;
- u8 numa_only;
+ bool use_custom_mac;
+ vl_api_mac_address_t mac_address;
+ vl_api_bond_mode_t mode;
+ vl_api_bond_lb_algo_t lb;
+ bool numa_only;
};
/** \brief Reply for bond create reply
@@ -52,7 +74,7 @@ define bond_create_reply
{
u32 context;
i32 retval;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
};
/** \brief Delete bond interface
@@ -64,7 +86,7 @@ autoreply define bond_delete
{
u32 client_index;
u32 context;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
};
/** \brief Initialize a new bond interface with the given paramters
@@ -79,10 +101,10 @@ define bond_enslave
{
u32 client_index;
u32 context;
- u32 sw_if_index;
- u32 bond_sw_if_index;
- u8 is_passive;
- u8 is_long_timeout;
+ vl_api_interface_index_t sw_if_index;
+ vl_api_interface_index_t bond_sw_if_index;
+ bool is_passive;
+ bool is_long_timeout;
};
/** \brief Reply for bond enslave reply
@@ -104,7 +126,7 @@ autoreply define bond_detach_slave
{
u32 client_index;
u32 context;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
};
/** \brief Dump bond interfaces request */
@@ -117,24 +139,24 @@ define sw_interface_bond_dump
/** \brief Reply for bond dump request
@param sw_if_index - software index of bond interface
@param id - ID of interface
- @param interface_name - name of interface
@param mode - bonding mode
@param lb - load balance algo
@param numa_only - enable local numa TX for lacp mode
@param active_slaves - active slaves count
@param slaves - config slave count
+ @param interface_name - name of interface
*/
define sw_interface_bond_details
{
u32 context;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
u32 id;
- u8 interface_name[64];
- u8 mode;
- u8 lb;
- u8 numa_only;
+ vl_api_bond_mode_t mode;
+ vl_api_bond_lb_algo_t lb;
+ bool numa_only;
u32 active_slaves;
u32 slaves;
+ string interface_name[64];
};
/** \brief bond slave dump
@@ -146,7 +168,7 @@ define sw_interface_slave_dump
{
u32 client_index;
u32 context;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
};
/** \brief Reply for slave dump request
@@ -160,11 +182,11 @@ define sw_interface_slave_dump
define sw_interface_slave_details
{
u32 context;
- u32 sw_if_index;
- u8 interface_name[64];
- u8 is_passive;
- u8 is_long_timeout;
- u8 is_local_numa;
+ vl_api_interface_index_t sw_if_index;
+ string interface_name[64];
+ bool is_passive;
+ bool is_long_timeout;
+ bool is_local_numa;
u32 weight;
};
@@ -178,7 +200,7 @@ autoreply define sw_interface_set_bond_weight
{
u32 client_index;
u32 context;
- u32 sw_if_index;
+ vl_api_interface_index_t sw_if_index;
u32 weight;
};
diff --git a/src/vnet/bonding/bond_api.c b/src/vnet/bonding/bond_api.c
index 74334b52bf2..ebbb80e956b 100644
--- a/src/vnet/bonding/bond_api.c
+++ b/src/vnet/bonding/bond_api.c
@@ -23,6 +23,7 @@
#include <vnet/interface.h>
#include <vnet/api_errno.h>
#include <vnet/ethernet/ethernet.h>
+#include <vnet/ethernet/ethernet_types_api.h>
#include <vnet/vnet_msg_enum.h>
@@ -78,12 +79,12 @@ vl_api_bond_create_t_handler (vl_api_bond_create_t * mp)
if (mp->use_custom_mac)
{
- clib_memcpy (ap->hw_addr, mp->mac_address, 6);
+ mac_address_decode (mp->mac_address, (mac_address_t *) ap->hw_addr);
ap->hw_addr_set = 1;
}
- ap->mode = mp->mode;
- ap->lb = mp->lb;
+ ap->mode = ntohl (mp->mode);
+ ap->lb = ntohl (mp->lb);
ap->numa_only = mp->numa_only;
bond_create_if (vm, ap);
@@ -168,8 +169,8 @@ bond_send_sw_interface_details (vpe_api_main_t * am,
clib_memcpy (mp->interface_name, bond_if->interface_name,
MIN (ARRAY_LEN (mp->interface_name) - 1,
strlen ((const char *) bond_if->interface_name)));
- mp->mode = bond_if->mode;
- mp->lb = bond_if->lb;
+ mp->mode = htonl (bond_if->mode);
+ mp->lb = htonl (bond_if->lb);
mp->numa_only = bond_if->numa_only;
mp->active_slaves = htonl (bond_if->active_slaves);
mp->slaves = htonl (bond_if->slaves);