devices: tap API cleanup
Use consistent API types.
Type: fix
Change-Id: I11cc7f6347b7a60e5fd41e54f0c7994e2d81199f
Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com>
diff --git a/src/vnet/devices/tap/tapv2_api.c b/src/vnet/devices/tap/tapv2_api.c
index af3c276..1f1e18a 100644
--- a/src/vnet/devices/tap/tapv2_api.c
+++ b/src/vnet/devices/tap/tapv2_api.c
@@ -25,6 +25,9 @@
#include <vnet/ethernet/ethernet.h>
#include <vnet/ip/ip.h>
+#include <vnet/ethernet/ethernet_types_api.h>
+#include <vnet/ip/ip_types_api.h>
+
#include <vnet/vnet_msg_enum.h>
#define vl_typedefs /* define message structures */
@@ -68,20 +71,28 @@
ap->id = ntohl (mp->id);
if (!mp->use_random_mac)
{
- clib_memcpy (ap->mac_addr, mp->mac_address, 6);
+ mac_address_decode (mp->mac_address, &ap->mac_addr);
ap->mac_addr_set = 1;
}
ap->rx_ring_sz = ntohs (mp->rx_ring_sz);
ap->tx_ring_sz = ntohs (mp->tx_ring_sz);
ap->sw_if_index = (u32) ~ 0;
+ if (mp->num_rx_queues < 1)
+ {
+ ap->rv = VNET_API_ERROR_INVALID_ARGUMENT;
+ ap->sw_if_index = ~0;
+ goto done;
+ }
+
+ ap->num_rx_queues = mp->num_rx_queues;
+
if (mp->host_if_name_set)
ap->host_if_name = mp->host_if_name;
if (mp->host_mac_addr_set)
{
- clib_memcpy (ap->host_mac_addr, mp->host_mac_addr, 6);
- ap->mac_addr_set = 1;
+ mac_address_decode (mp->host_mac_addr, &ap->host_mac_addr);
}
if (mp->host_namespace_set)
@@ -90,27 +101,27 @@
if (mp->host_bridge_set)
ap->host_bridge = mp->host_bridge;
- if (mp->host_ip4_addr_set)
+ if (mp->host_ip4_prefix_set)
{
- clib_memcpy (&ap->host_ip4_addr.as_u8, mp->host_ip4_addr, 4);
- ap->host_ip4_prefix_len = mp->host_ip4_prefix_len;
+ ip4_address_decode (mp->host_ip4_prefix.address, &ap->host_ip4_addr);
+ ap->host_ip4_prefix_len = mp->host_ip4_prefix.len;
}
- if (mp->host_ip6_addr_set)
+ if (mp->host_ip6_prefix_set)
{
- clib_memcpy (&ap->host_ip6_addr, mp->host_ip6_addr, 16);
- ap->host_ip6_prefix_len = mp->host_ip6_prefix_len;
+ ip6_address_decode (mp->host_ip6_prefix.address, &ap->host_ip6_addr);
+ ap->host_ip6_prefix_len = mp->host_ip6_prefix.len;
}
if (mp->host_ip4_gw_set)
{
- clib_memcpy (&ap->host_ip4_gw, mp->host_ip4_gw, 4);
+ ip4_address_decode (mp->host_ip4_gw, &ap->host_ip4_gw);
ap->host_ip4_gw_set = 1;
}
if (mp->host_ip6_gw_set)
{
- clib_memcpy (&ap->host_ip6_gw, mp->host_ip6_gw, 16);
+ ip6_address_decode (mp->host_ip6_gw, &ap->host_ip6_gw);
ap->host_ip6_gw_set = 1;
}
@@ -126,15 +137,13 @@
/* If a tag was supplied... */
- if (mp->tag[0])
+ if (vl_api_string_len (&mp->tag))
{
- /* Make sure it's a proper C-string */
- mp->tag[ARRAY_LEN (mp->tag) - 1] = 0;
- u8 *tag = format (0, "%s%c", mp->tag, 0);
+ u8 *tag = format (0, "%s%c", vl_api_from_api_string (&mp->tag), 0);
vnet_set_sw_interface_tag (vnm, tag, ap->sw_if_index);
}
-
+done:
rmp = vl_msg_api_alloc (sizeof (*rmp));
rmp->_vl_msg_id = ntohs (VL_API_TAP_CREATE_V2_REPLY);
rmp->context = mp->context;
@@ -191,7 +200,7 @@
strlen ((const char *) tap_if->dev_name)));
mp->rx_ring_sz = htons (tap_if->rx_ring_sz);
mp->tx_ring_sz = htons (tap_if->tx_ring_sz);
- clib_memcpy (mp->host_mac_addr, tap_if->host_mac_addr, 6);
+ mac_address_encode (&tap_if->host_mac_addr, mp->host_mac_addr);
clib_memcpy (mp->host_if_name, tap_if->host_if_name,
MIN (ARRAY_LEN (mp->host_if_name) - 1,
strlen ((const char *) tap_if->host_if_name)));
@@ -202,12 +211,14 @@
MIN (ARRAY_LEN (mp->host_bridge) - 1,
strlen ((const char *) tap_if->host_bridge)));
mp->host_mtu_size = htonl (tap_if->host_mtu_size);
+ mac_address_encode (&tap_if->host_mac_addr, mp->host_mac_addr);
+
if (tap_if->host_ip4_prefix_len)
- clib_memcpy (&mp->host_ip4_addr, &tap_if->host_ip4_addr, 4);
- mp->host_ip4_prefix_len = tap_if->host_ip4_prefix_len;
+ ip4_address_encode (&tap_if->host_ip4_addr, mp->host_ip4_prefix.address);
+ mp->host_ip4_prefix.len = tap_if->host_ip4_prefix_len;
if (tap_if->host_ip6_prefix_len)
- clib_memcpy (&mp->host_ip6_addr, &tap_if->host_ip6_addr, 16);
- mp->host_ip6_prefix_len = tap_if->host_ip6_prefix_len;
+ ip6_address_encode (&tap_if->host_ip6_addr, mp->host_ip6_prefix.address);
+ mp->host_ip6_prefix.len = tap_if->host_ip6_prefix_len;
mp->context = context;
vl_api_send_msg (reg, (u8 *) mp);
@@ -222,11 +233,16 @@
vl_api_registration_t *reg;
tap_interface_details_t *tapifs = NULL;
tap_interface_details_t *tap_if = NULL;
+ u32 filter_sw_if_index;
reg = vl_api_client_index_to_registration (mp->client_index);
if (!reg)
return;
+ filter_sw_if_index = htonl (mp->sw_if_index);
+ if (filter_sw_if_index != ~0)
+ return; /* UNIMPLEMENTED */
+
rv = tap_dump_ifs (&tapifs);
if (rv)
return;